Add Who::Request::GetFlagIndex to get field index

Replaces the dirty logic in m_hideoper and m_namesx

1 files changed, 3 insertions, 16 deletions

diff --git a/src/modules/m_hideoper.cpp b/src/modules/m_hideoper.cpp
index f04d88809..8feb1a852 100644
--- a/src/modules/m_hideoper.cpp
+++ b/src/modules/m_hideoper.cpp
@@ -123,22 +123,9 @@ class ModuleHideOper
 			if (request.flags['o'])
 				return MOD_RES_DENY;
 
-			size_t flag_index = 5;
-			if (request.whox)
-			{
-				// We only need to fiddle with the flags if they are present.
-				if (!request.whox_fields['f'])
-					return MOD_RES_PASSTHRU;
-
-				// WHOX makes this a bit tricky as we need to work out the parameter which the flags are in.
-				flag_index = 0;
-				static const char* flags = "tcuihsn";
-				for (size_t i = 0; i < strlen(flags); ++i)
-				{
-					if (request.whox_fields[flags[i]])
-						flag_index += 1;
-				}
-			}
+			size_t flag_index;
+			if (!request.GetFlagIndex('f', flag_index))
+				return MOD_RES_PASSTHRU;
 
 			// hide the "*" that marks the user as an oper from the /WHO line
 			// #chan ident localhost insp22.test nick H@ :0 Attila