Only assign NewServices once the duplicate check is done.

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

diff --git a/src/modules/m_geoclass.cpp b/src/modules/m_geoclass.cpp
index a1807ae00db431b62c746c26b13dd6d75c87dd93..8289c9a6014cbcabdc99ee5bf3aff14ae86ff9f2 100644 (file)
--- a/src/modules/m_geoclass.cpp
+++ b/src/modules/m_geoclass.cpp
@@ -43,7 +43,7 @@ class ModuleGeoClass
 
        Version GetVersion() CXX11_OVERRIDE
        {
-               return Version("Provides a way to assign users to connect classes by country", VF_VENDOR);
+               return Version("Allows the server administrator to assign users to connect classes by the country they are connecting from.", VF_VENDOR);
        }
 
        ModResult OnSetConnectClass(LocalUser* user, ConnectClass* myclass) CXX11_OVERRIDE
@@ -68,6 +68,8 @@ class ModuleGeoClass
 
                // A list of country codes were specified but the user didn't match
                // any of them.
+               ServerInstance->Logs->Log("CONNECTCLASS", LOG_DEBUG, "The %s connect class is not suitable as the origin country (%s) is not any of %s",
+                       myclass->GetName().c_str(), code.c_str(), country.c_str());
                return MOD_RES_DENY;
        }