Oops, actually, return 1 to block change, 0 to check it, -1 to allow it regardless...

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@10083 e03df62e-2008-0410-955e-edbf42e46eb7

diff --git a/include/modules.h b/include/modules.h
index 772f4ab5f57310d9269ebc65e00c22db22c70f5d..3f4be033a9e2fceff3627ee542695ab104fbe8b8 100644 (file)
--- a/include/modules.h
+++ b/include/modules.h
@@ -1173,7 +1173,7 @@ class CoreExport Module : public Extensible
        virtual int OnChangeLocalUserGECOS(User* user, const std::string &newhost); 
 
        /** Called whenever a topic is changed by a local user.
-        * Return 1 to deny the topic change, or 0 to allow it.
+        * Return 1 to deny the topic change, 0 to check details on the change, -1 to let it through with no checks
         * @param user The user changing the topic
         * @param chan The channels who's topic is being changed
         * @param topic The actual topic text

diff --git a/src/channels.cpp b/src/channels.cpp
index 19cfccc79c3f15903701001a38fb97bdddf6e910..a0ba81a5fb8fc835fc399453915600292fc4f0b2 100644 (file)
--- a/src/channels.cpp
+++ b/src/channels.cpp
@@ -81,7 +81,7 @@ int Channel::SetTopic(User *u, std::string &ntopic)
                /* 0: check status, 1: don't, -1: disallow change silently */
 
                FOREACH_RESULT(I_OnLocalTopicChange,OnLocalTopicChange(u,this,ntopic));
-               if (MOD_RESULT == -1)
+               if (MOD_RESULT == 1)
                        return CMD_FAILURE;
                else if (MOD_RESULT == 0)
                {