API header and client module updates for new multi-parameter query request. Needs...

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

diff --git a/src/modules/m_cban.cpp b/src/modules/m_cban.cpp
index 8739fc98ce6276752b8a7c694702267e093f2487..57bb9490a20c73749d069e73edea7ebad954252d 100644 (file)
--- a/src/modules/m_cban.cpp
+++ b/src/modules/m_cban.cpp
@@ -1,16 +1,16 @@
-/*       +------------------------------------+
- *       | Inspire Internet Relay Chat Daemon |
- *       +------------------------------------+
+/*   +------------------------------------+
+ *   | Inspire Internet Relay Chat Daemon |
+ *   +------------------------------------+
  *
  *  InspIRCd is copyright (C) 2002-2006 ChatSpike-Dev.
- *                       E-mail:
- *                <brain@chatspike.net>
- *               <Craig@chatspike.net>
+ *   E-mail:
+ *<brain@chatspike.net>
+ *       <Craig@chatspike.net>
  *                                             <omster@gmail.com>
- *     
+ * 
  * Written by Craig Edwards, Craig McLure, and others.
  * This program is free but copyrighted software; see
- *            the file COPYING for details.
+ *the file COPYING for details.
  *
  * ---------------------------------------------------
  */
@@ -27,7 +27,7 @@
 
 /* $ModDesc: Gives /cban, aka C:lines. Think Q:lines, for channels. */
 
-class CBan
+class CBan : public classbase
 {
 public:
        irc::string chname;
@@ -49,7 +49,6 @@ std::string EncodeCBan(const CBan &ban);
 CBan DecodeCBan(const std::string &data);
 bool CBanComp(const CBan &ban1, const CBan &ban2);
 void ExpireBans();
-bool IsValidChan(const char* cname);
 
 extern time_t TIME;
 typedef std::vector<CBan> cbanlist;
@@ -69,7 +68,7 @@ class cmd_cban : public command_t
                this->Srv = Me;
        }
 
-       void Handle(char **parameters, int pcnt, userrec *user)
+       void Handle(const char** parameters, int pcnt, userrec *user)
        {
                /* syntax: CBAN #channel time :reason goes here */
                /* 'time' is a human-readable timestring, like 2d3h2s. */
@@ -93,7 +92,7 @@ class cmd_cban : public command_t
                else if (pcnt >= 2)
                {
                        /* full form to add a CBAN */
-                       if(IsValidChan(parameters[0]))
+                       if (IsValidChannelName(parameters[0]))
                        {
                                // parameters[0] = #channel
                                // parameters[1] = 1h3m2s
@@ -118,7 +117,7 @@ class cmd_cban : public command_t
                        }
                        else
                        {
-                               WriteServ(user->fd, "403 %s %s :No such channel", user->nick, parameters[0]);
+                               WriteServ(user->fd, "403 %s %s :Invalid channel name", user->nick, parameters[0]);
                        }
                }
        }
@@ -165,7 +164,7 @@ class ModuleCBan : public Module
                /* check cbans in here, and apply as necessary. */
                for(cbanlist::iterator iter = cbans.begin(); iter != cbans.end(); iter++)
                {
-                       if(iter->chname == cname && !strchr(user->modes, 'o'))
+                       if(iter->chname == cname && !user->modes[UM_OPERATOR])
                        {
                                // Channel is banned.
                                WriteServ(user->fd, "384 %s %s :Cannot join channel, CBANed (%s)", user->nick, cname, iter->reason.c_str());
@@ -184,7 +183,7 @@ class ModuleCBan : public Module
                }
        }
        
-       virtual void OnDecodeMetaData(int target_type, void* target, std::string extname, std::string extdata)
+       virtual void OnDecodeMetaData(int target_type, void* target, const std::string &extname, const std::string &extdata)
        {
                if((target_type == TYPE_OTHER) && (extname == "cban"))
                {
@@ -199,29 +198,26 @@ class ModuleCBan : public Module
        
        virtual Version GetVersion()
        {
-               return Version(1,0,0,0,VF_VENDOR);
+               return Version(1,0,0,1,VF_VENDOR);
        }
 };
 
 std::string EncodeCBan(const CBan &ban)
 {
        std::ostringstream stream;      
-       stream << ban.chname.c_str() << " " << ban.set_by << " " << ban.set_on << " " << ban.length << " " << ban.reason;
+       stream << ban.chname << " " << ban.set_by << " " << ban.set_on << " " << ban.length << " " << ban.reason;
        return stream.str();    
 }
 
 CBan DecodeCBan(const std::string &data)
 {
        CBan res;
-       std::istringstream stream;
-       // XXX - Change this...we shouldn't need tempname, need an overloaded iostream operator on irc::string?
-       std::string tempname;
-       stream >> tempname;
+       std::istringstream stream(data);
+       stream >> res.chname;
        stream >> res.set_by;
        stream >> res.set_on;
        stream >> res.length;
        res.reason = stream.str();
-       res.chname = tempname.c_str();
        
        return res;
 }
@@ -233,29 +229,30 @@ bool CBanComp(const CBan &ban1, const CBan &ban2)
 
 void ExpireBans()
 {
-       while(cbans.size() && ((cbans.begin()->set_on + cbans.begin()->length) <= TIME))
+       bool go_again = true;
+
+       while (go_again)
        {
-               cbanlist::iterator iter = cbans.begin();
-               
-               log(DEBUG, "m_cban.so: Ban on %s expired, removing...", iter->chname.c_str());
-               WriteOpers("*** %li second CBAN on %s (%s) set %u seconds ago expired", iter->length, iter->chname.c_str(), iter->reason.c_str(), TIME - iter->set_on);
-               cbans.erase(iter);
-       }
-}
+               go_again = false;
 
-bool IsValidChan(const char* cname)
-{
-       if(!cname)
-               return false;
-               
-       if(cname[0] != '#')
-               return false;
-               
-       for(unsigned int i = 0; i < strlen(cname); i++)
-               if((cname[i] == ' ') || (cname[i] == '\7') || (cname[i] == ','))
-                       return false;
-                       
-       return true;
+               for (cbanlist::iterator iter = cbans.begin(); iter != cbans.end(); iter++)
+               {
+                       /* 0 == permanent, don't mess with them! -- w00t */
+                       if (iter->length != 0)
+                       {
+                               if (iter->set_on + iter->length <= TIME)
+                               {
+                                       log(DEBUG, "m_cban.so: Ban on %s expired, removing...", iter->chname.c_str());
+                                       WriteOpers("*** %li second CBAN on %s (%s) set %u seconds ago expired", iter->length, iter->chname.c_str(), iter->reason.c_str(), TIME - iter->set_on);
+                                       cbans.erase(iter);
+                                       go_again = true;
+                               }
+                       }
+
+                       if (go_again == true)
+                               break;
+               }
+       }
 }
 
 class ModuleCBanFactory : public ModuleFactory