Move all the parameters around AGAIN. See docs shortly

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

diff --git a/src/modules/m_abbreviation.cpp b/src/modules/m_abbreviation.cpp
index f94f8a5cc7133e02d9c97390437e3de61b09dbb3..3bc4c919344d6a75847b48259f60819bee377859 100644 (file)
--- a/src/modules/m_abbreviation.cpp
+++ b/src/modules/m_abbreviation.cpp
@@ -19,7 +19,7 @@ class ModuleAbbreviation : public Module
 {
 
  public:
-       
+
        ModuleAbbreviation(InspIRCd* Me)
                : Module(Me)
        {
@@ -30,7 +30,7 @@ class ModuleAbbreviation : public Module
 
        virtual Version GetVersion()
        {
-               return Version(1,2,0,0,VF_VENDOR,API_VERSION);
+               return Version("$Id$",VF_VENDOR,API_VERSION);
        }
 
        virtual int OnPreCommand(std::string &command, std::vector<std::string> &parameters, User *user, bool validated, const std::string &original_line)
@@ -44,6 +44,8 @@ class ModuleAbbreviation : public Module
 
                /* Look for any command that starts with the same characters, if it does, replace the command string with it */
                size_t clen = command.length();
+               std::string foundcommand, matchlist;
+               bool foundmatch = false;
                for (Commandtable::iterator n = ServerInstance->Parser->cmdlist.begin(); n != ServerInstance->Parser->cmdlist.end(); ++n)
                {
                        if (n->first.length() < clen)
@@ -51,14 +53,40 @@ class ModuleAbbreviation : public Module
 
                        if (command == n->first.substr(0, clen))
                        {
-                               /* Found the command */
-                               command = n->first;
-                               return false;
+                               if (matchlist.length() > 450)
+                               {
+                                       user->WriteNumeric(420, "%s :Ambiguous abbreviation and too many possible matches.", user->nick.c_str());
+                                       return true;
+                               }
+
+                               if (!foundmatch)
+                               {
+                                       /* Found the command */
+                                       foundcommand = n->first;
+                                       foundmatch = true;
+                               }
+                               else
+                                       matchlist.append(" ").append(n->first);
                        }
                }
 
-               /* No match, we have to put the . back again so that the invalid command numeric looks correct. */
-               command += '.';
+               /* Ambiguous command, list the matches */
+               if (!matchlist.empty())
+               {
+                       user->WriteNumeric(420, "%s :Ambiguous abbreviation, posssible matches: %s%s", user->nick.c_str(), foundcommand.c_str(), matchlist.c_str());
+                       return true;
+               }
+
+               if (foundcommand.empty())
+               {
+                       /* No match, we have to put the . back again so that the invalid command numeric looks correct. */
+                       command += '.';
+               }
+               else
+               {
+                       command = foundcommand;
+               }
+
                return false;
        }
 };