X-Git-Url: https://git.netwichtig.de/gitweb/?a=blobdiff_plain;f=src%2Fmode.cpp;h=86438bf733fce705c18b9933e1d3546fbb563c91;hb=1524caf2f799cff54c2de330c9670a0b761ba3d8;hp=c760df3f5c93f9940cce104feb9cba81c0c0b863;hpb=b4be0c94ab5fb7e5a7a799a195c78de072a5e315;p=user%2Fhenk%2Fcode%2Finspircd.git

diff --git a/src/mode.cpp b/src/mode.cpp
index c760df3f5..86438bf73 100644
--- a/src/mode.cpp
+++ b/src/mode.cpp
@@ -49,8 +49,11 @@
 /* +s (server notice masks) */
 #include "modes/umode_s.h"
 
-ModeHandler::ModeHandler(InspIRCd* Instance, Module* Creator, char modeletter, int parameters_on, int parameters_off, bool listmode, ModeType type, bool operonly, char mprefix, char prefixrequired, TranslateType translate)
-	: ServerInstance(Instance), mode(modeletter), n_params_on(parameters_on), n_params_off(parameters_off), list(listmode), m_type(type), m_paramtype(translate), oper(operonly), prefix(mprefix), count(0), prefixneeded(prefixrequired), creator(Creator)
+InspIRCd* ModeHandler::ServerInstance;
+
+ModeHandler::ModeHandler(Module* Creator, char modeletter, ParamSpec Params, ModeType type)
+	: mode(modeletter), parameters_taken(Params), list(false), m_type(type), m_paramtype(TR_TEXT),
+	oper(false), prefix(0), count(0), levelrequired(HALFOP_VALUE), creator(Creator)
 {
 }
 
@@ -63,18 +66,6 @@ bool ModeHandler::IsListMode()
 	return list;
 }
 
-char ModeHandler::GetNeededPrefix()
-{
-	if (prefixneeded == '%' && !ServerInstance->Config->AllowHalfop)
-		return '@';
-	return prefixneeded;
-}
-
-void ModeHandler::SetNeededPrefix(char needsprefix)
-{
-	prefixneeded = needsprefix;
-}
-
 unsigned int ModeHandler::GetPrefixRank()
 {
 	return 0;
@@ -91,29 +82,18 @@ void ModeHandler::ChangeCount(int modifier)
 	ServerInstance->Logs->Log("MODE", DEBUG,"Change count for mode %c is now %d", mode, count);
 }
 
-ModeType ModeHandler::GetModeType()
-{
-	return m_type;
-}
-
-TranslateType ModeHandler::GetTranslateType()
-{
-	return m_paramtype;
-}
-
-bool ModeHandler::NeedsOper()
-{
-	return oper;
-}
-
-char ModeHandler::GetPrefix()
-{
-	return prefix;
-}
-
 int ModeHandler::GetNumParams(bool adding)
 {
-	return adding ? n_params_on : n_params_off;
+	switch (parameters_taken)
+	{
+		case PARAM_ALWAYS:
+			return 1;
+		case PARAM_SETONLY:
+			return adding ? 1 : 0;
+		case PARAM_NONE:
+			break;
+	}
+	return 0;
 }
 
 char ModeHandler::GetModeChar()
@@ -292,36 +272,21 @@ ModeAction ModeParser::TryMode(User* user, User* targetuser, Channel* chan, bool
 
 	if (chan && !SkipACL && (MOD_RESULT != MOD_RES_ALLOW))
 	{
-		char needed = mh->GetNeededPrefix();
-		ModeHandler* prefixmode = FindPrefix(needed);
-
-		/* If the mode defined by the handler is not '\0', but the handler for it
-		 * cannot be found, they probably dont have the right module loaded to implement
-		 * the prefix they want to compare the mode against, e.g. '&' for m_chanprotect.
-		 * Revert to checking against the minimum core prefix, '%'.
+		unsigned int neededrank = mh->GetLevelRequired();
+		/* Compare our rank on the channel against the rank of the required prefix,
+		 * allow if >= ours. Because mIRC and xchat throw a tizz if the modes shown
+		 * in NAMES(X) are not in rank order, we know the most powerful mode is listed
+		 * first, so we don't need to iterate, we just look up the first instead.
 		 */
-		if (needed && !prefixmode)
-		{
-			needed = ServerInstance->Config->AllowHalfop ? '%' : '@';
-			prefixmode = FindPrefix(needed);
-		}
-
-		if (needed)
-		{
-			unsigned int neededrank = prefixmode->GetPrefixRank();
-			/* Compare our rank on the channel against the rank of the required prefix,
-			 * allow if >= ours. Because mIRC and xchat throw a tizz if the modes shown
-			 * in NAMES(X) are not in rank order, we know the most powerful mode is listed
-			 * first, so we don't need to iterate, we just look up the first instead.
-			 */
-			unsigned int ourrank = chan->GetPrefixValue(user);
-			if (ourrank < neededrank)
-			{
-				/* Bog off */
-				user->WriteNumeric(ERR_CHANOPRIVSNEEDED, "%s %s :You must have channel privilege %c or above to %sset channel mode %c",
-						user->nick.c_str(), chan->name.c_str(), needed, adding ? "" : "un", modechar);
-				return MODEACTION_DENY;
-			}
+		unsigned int ourrank = chan->GetPrefixValue(user);
+		if (ourrank < neededrank)
+		{
+			/* Bog off */
+			// TODO replace with a real search for the proper prefix
+			char needed = neededrank > HALFOP_VALUE ? '@' : '%';
+			user->WriteNumeric(ERR_CHANOPRIVSNEEDED, "%s %s :You must have channel privilege %c or above to %sset channel mode %c",
+					user->nick.c_str(), chan->name.c_str(), needed, adding ? "" : "un", modechar);
+			return MODEACTION_DENY;
 		}
 	}
 
@@ -378,7 +343,7 @@ ModeAction ModeParser::TryMode(User* user, User* targetuser, Channel* chan, bool
 	{
 		User* user_to_prefix = ServerInstance->FindNick(parameter);
 		if (user_to_prefix)
-			chan->SetPrefix(user_to_prefix, modechar, mh->GetPrefixRank(), adding);
+			chan->SetPrefix(user_to_prefix, modechar, adding);
 	}
 
 	for (ModeWatchIter watchers = modewatchers[handler_id].begin(); watchers != modewatchers[handler_id].end(); watchers++)
@@ -419,13 +384,8 @@ void ModeParser::Process(const std::vector<std::string>& parameters, User *user,
 	}
 	else
 	{
-		/* Overall access control hook for mode change */
-		int hook = targetchannel ? AC_GENERAL_MODE : AC_GENERAL_UMODE;
-
-		LastParse = mode_sequence;
 		ModResult MOD_RESULT;
-		FIRST_MOD_RESULT(ServerInstance, OnAccessCheck, MOD_RESULT, (user, targetuser, targetchannel, hook));
-		LastParse.clear();
+		FIRST_MOD_RESULT(ServerInstance, OnPreMode, MOD_RESULT, (user, targetuser, targetchannel, parameters));
 		if (MOD_RESULT == MOD_RES_DENY)
 			return;
 		SkipAccessChecks = (MOD_RESULT == MOD_RES_ALLOW);
@@ -983,6 +943,7 @@ void ModeHandler::RemoveMode(Channel* channel, irc::modestacker* stack)
 
 ModeParser::ModeParser(InspIRCd* Instance) : ServerInstance(Instance)
 {
+	ModeHandler::ServerInstance = Instance;
 	ModeHandler* modes[] =
 	{
 		new ModeChannelSecret(Instance),