Fix a typo in 327fe7c than changed the logic of a function and simplify IsXXXAccount functions

1 files changed, 5 insertions, 44 deletions

diff --git a/src/server/game/Accounts/AccountMgr.cpp b/src/server/game/Accounts/AccountMgr.cpp
index c8870a82966..f2478ae4a6c 100755
--- a/src/server/game/Accounts/AccountMgr.cpp
+++ b/src/server/game/Accounts/AccountMgr.cpp
@@ -222,66 +222,27 @@ std::string CalculateShaPassHash(std::string& name, std::string& password)
 
 bool IsPlayerAccount(uint32 gmlevel)
 {
-    switch (gmlevel)
-    {
-        case SEC_CONSOLE:
-        case SEC_ADMINISTRATOR:
-        case SEC_GAMEMASTER:
-        case SEC_MODERATOR:
-            return false;
-        default:
-            return true;
-    }
+    return gmlevel = SEC_PLAYER;
 }
 
 bool IsModeratorAccount(uint32 gmlevel)
 {
-    switch (gmlevel)
-    {
-        case SEC_CONSOLE:
-        case SEC_ADMINISTRATOR:
-        case SEC_GAMEMASTER:
-        case SEC_MODERATOR:
-            return true;
-        default:
-            return false;
-    }
+    return gmlevel >= SEC_MODERATOR && gmlevel <= SEC_CONSOLE;
 }
 
 bool IsGMAccount(uint32 gmlevel)
 {
-    switch (gmlevel)
-    {
-        case SEC_CONSOLE:
-        case SEC_ADMINISTRATOR:
-        case SEC_GAMEMASTER:
-            return true;
-        default:
-            return false;
-    }
+    return gmlevel >= SEC_GAMEMASTER && gmlevel <= SEC_CONSOLE;
 }
 
 bool IsAdminAccount(uint32 gmlevel)
 {
-    switch (gmlevel)
-    {
-        case SEC_CONSOLE:
-        case SEC_ADMINISTRATOR:
-            return true;
-        default:
-            return false;
-    }
+    return gmlevel >= SEC_ADMINISTRATOR && gmlevel <= SEC_CONSOLE;
 }
 
 bool IsConsoleAccount(uint32 gmlevel)
 {
-    switch (gmlevel)
-    {
-        case SEC_CONSOLE:
-            return true;
-        default:
-            return false;
-    }
+    return gmlevel = SEC_CONSOLE;
 }
 
 } // Namespace AccountMgr