Allow multiple autoconnects in a single <autoconnect> tag, fix infinite failover

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

diff --git a/src/modules/m_allowinvite.cpp b/src/modules/m_allowinvite.cpp
index ce4d73098e331b0831d0846a7a8090b35626063c..0de9925015184107406cd975a44b98344a3f0c09 100644 (file)
--- a/src/modules/m_allowinvite.cpp
+++ b/src/modules/m_allowinvite.cpp
@@ -18,7 +18,7 @@
 class AllowInvite : public SimpleChannelModeHandler
 {
  public:
-       AllowInvite(InspIRCd* Instance, Module* Creator) : SimpleChannelModeHandler(Instance, Creator, 'A') { }
+       AllowInvite(Module* Creator) : SimpleChannelModeHandler(Creator, 'A') { }
 };
 
 class ModuleAllowInvite : public Module
@@ -26,7 +26,7 @@ class ModuleAllowInvite : public Module
        AllowInvite ni;
  public:
 
-       ModuleAllowInvite(InspIRCd* Me) : Module(Me), ni(Me, this)
+       ModuleAllowInvite() : ni(this)
        {
                if (!ServerInstance->Modes->AddMode(&ni))
                        throw ModuleException("Could not add new modes!");
@@ -39,24 +39,25 @@ class ModuleAllowInvite : public Module
                ServerInstance->AddExtBanChar('A');
        }
 
-       virtual int OnUserPreInvite(User* user,User* dest,Channel* channel, time_t timeout)
+       virtual ModResult OnUserPreInvite(User* user,User* dest,Channel* channel, time_t timeout)
        {
                if (IS_LOCAL(user))
                {
-                       if (channel->GetExtBanStatus(user, 'A') == -1)
+                       ModResult res = channel->GetExtBanStatus(user, 'A');
+                       if (res == MOD_RES_DENY)
                        {
                                // Matching extban, explicitly deny /invite
                                user->WriteNumeric(ERR_CHANOPRIVSNEEDED, "%s %s :You are banned from using INVITE", user->nick.c_str(), channel->name.c_str());
-                               return 1;
+                               return res;
                        }
-                       if (channel->IsModeSet('A') || channel->GetExtBanStatus(user, 'A') == 1)
+                       if (channel->IsModeSet('A') || res == MOD_RES_ALLOW)
                        {
                                // Explicitly allow /invite
-                               return -1;
+                               return MOD_RES_ALLOW;
                        }
                }
 
-               return 0;
+               return MOD_RES_PASSTHRU;
        }
 
        virtual ~ModuleAllowInvite()
@@ -66,7 +67,7 @@ class ModuleAllowInvite : public Module
 
        virtual Version GetVersion()
        {
-               return Version("$Id$",VF_COMMON|VF_VENDOR,API_VERSION);
+               return Version("Provides support for channel mode +A, allowing /invite freely on a channel (and extban A to allow specific users it)",VF_COMMON|VF_VENDOR,API_VERSION);
        }
 };