Check the unset level in autoop as well as the set level.

[user/henk/code/inspircd.git] / src / modules / m_commonchans.cpp

diff --git a/src/modules/m_commonchans.cpp b/src/modules/m_commonchans.cpp
index 94cea613b066341e5e2f26ff2847540c47dbf8bb..5a30ddddb958bcda26e6016055261a9b89eb17c5 100644 (file)
--- a/src/modules/m_commonchans.cpp
+++ b/src/modules/m_commonchans.cpp
@@ -28,19 +28,19 @@ class ModuleCommonChans
  private:
        SimpleUserModeHandler mode;
 
-       ModResult HandleMessage(User* user, const MessageTarget& target) CXX11_OVERRIDE
+       ModResult HandleMessage(User* user, const MessageTarget& target)
        {
                if (target.type != MessageTarget::TYPE_USER)
                        return MOD_RES_PASSTHRU;
 
                User* targuser = target.Get<User>();
-               if (!targuser->IsModeSet(mode) || !user->SharesChannelWith(targuser))
+               if (!targuser->IsModeSet(mode) || user->SharesChannelWith(targuser))
                        return MOD_RES_PASSTHRU;
 
                if (user->HasPrivPermission("users/ignore-commonchans") || user->server->IsULine())
                        return MOD_RES_PASSTHRU;
 
-               user->WriteNumeric(ERR_CANTSENDTOUSER, targuser->nick, "You are not permitted to send private messages to this user (+c set)");
+               user->WriteNumeric(ERR_CANTSENDTOUSER, targuser->nick, "You are not permitted to send private messages to this user (+c is set)");
                return MOD_RES_DENY;
        }
 
@@ -53,7 +53,7 @@ class ModuleCommonChans
 
        Version GetVersion() CXX11_OVERRIDE
        {
-               return Version("Adds user mode +c which requires users to share a common channel with you to private message you", VF_VENDOR);
+               return Version("Provides user mode +c, requires users to share a common channel with you to private message you", VF_VENDOR);
        }
 
        ModResult OnUserPreMessage(User* user, const MessageTarget& target, MessageDetails& details) CXX11_OVERRIDE