Add {To,From}{Human,Internal,Network} to ExtensionItem.

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

diff --git a/src/modules/m_anticaps.cpp b/src/modules/m_anticaps.cpp
index aa605e6edae8154c108d7d7d8733af44aedcd286..8f020ed03d1410d9319025bff2c084b5d9fbee45 100644 (file)
--- a/src/modules/m_anticaps.cpp
+++ b/src/modules/m_anticaps.cpp
@@ -34,9 +34,9 @@ class AntiCapsSettings
  public:
        const AntiCapsMethod method;
        const uint16_t minlen;
-       const uint16_t percent;
+       const uint8_t percent;
 
-       AntiCapsSettings(const AntiCapsMethod& Method, const uint16_t& MinLen, const uint16_t& Percent)
+       AntiCapsSettings(const AntiCapsMethod& Method, const uint16_t& MinLen, const uint8_t& Percent)
                : method(Method)
                , minlen(MinLen)
                , percent(Percent)
@@ -83,13 +83,13 @@ class AntiCapsMode : public ParamMode<AntiCapsMode, SimpleExtItem<AntiCapsSettin
                return true;
        }
 
-       bool ParsePercent(irc::sepstream& stream, uint16_t& percent)
+       bool ParsePercent(irc::sepstream& stream, uint8_t& percent)
        {
                std::string percentstr;
                if (!stream.GetToken(percentstr))
                        return false;
 
-               uint16_t result = ConvToNum<uint16_t>(percentstr);
+               uint8_t result = ConvToNum<uint8_t>(percentstr);
                if (result < 1 || result > 100)
                        return false;
 
@@ -101,6 +101,7 @@ class AntiCapsMode : public ParamMode<AntiCapsMode, SimpleExtItem<AntiCapsSettin
        AntiCapsMode(Module* Creator)
                : ParamMode<AntiCapsMode, SimpleExtItem<AntiCapsSettings> >(Creator, "anticaps", 'B')
        {
+               syntax = "{ban|block|mute|kick|kickban}:<minlen>:<percent>";
        }
 
        ModeAction OnSet(User* source, Channel* channel, std::string& parameter) CXX11_OVERRIDE
@@ -108,12 +109,12 @@ class AntiCapsMode : public ParamMode<AntiCapsMode, SimpleExtItem<AntiCapsSettin
                irc::sepstream stream(parameter, ':');
                AntiCapsMethod method;
                uint16_t minlen;
-               uint16_t percent;
+               uint8_t percent;
 
                // Attempt to parse the method.
                if (!ParseMethod(stream, method) || !ParseMinimumLength(stream, minlen) || !ParsePercent(stream, percent))
                {
-                       source->WriteNumeric(Numerics::InvalidModeParameter(channel, this, parameter, "Invalid anticaps mode parameter. Syntax: <ban|block|mute|kick|kickban>:{minlen}:{percent}."));
+                       source->WriteNumeric(Numerics::InvalidModeParameter(channel, this, parameter));
                        return MODEACTION_DENY;
                }
 
@@ -148,7 +149,7 @@ class AntiCapsMode : public ParamMode<AntiCapsMode, SimpleExtItem<AntiCapsSettin
                out.push_back(':');
                out.append(ConvToStr(acs->minlen));
                out.push_back(':');
-               out.append(ConvToStr(acs->percent));
+               out.append(ConvNumeric(acs->percent));
        }
 };
 
@@ -173,7 +174,7 @@ class ModuleAntiCaps : public Module
 
        void InformUser(Channel* channel, User* user, const std::string& message)
        {
-               user->WriteNumeric(ERR_CANNOTSENDTOCHAN, channel, message + " and was blocked.");
+               user->WriteNumeric(ERR_CANNOTSENDTOCHAN, channel->name, message + " and was blocked.");
        }
 
  public:
@@ -297,7 +298,7 @@ class ModuleAntiCaps : public Module
 
        Version GetVersion() CXX11_OVERRIDE
        {
-               return Version("Provides support for punishing users that send capitalised messages.", VF_COMMON|VF_VENDOR);
+               return Version("Provides support for punishing users that send capitalised messages", VF_COMMON|VF_VENDOR);
        }
 };