ChaosMod: Improve effect condition logic

Author: Regynate

ChaosMod/Components/EffectDispatcher.cpp

@@ -538,9 +538,9 @@ void EffectDispatcher::DispatchRandomEffect(DispatchEffectFlags dispatchEffectFl
 		return;
 
 	std::unordered_map<EffectIdentifier, EffectData, EffectsIdentifierHasher> choosableEffects;
-	for (const auto &[effectId, effectData] : g_EnabledEffects)
-		if (!effectData.IsMeta() && !effectData.IsUtility() && !effectData.IsHidden())
-			choosableEffects.emplace(effectId, effectData);
+	for (const auto &effectData : GetFilteredEnabledEffects())
+		if (!effectData->IsMeta() && !effectData->IsUtility() && !effectData->IsHidden())
+			choosableEffects.emplace(effectData->Id, *effectData);
 
 	float totalWeight = 0.f;
 	for (const auto &[effectId, effectData] : choosableEffects)

ChaosMod/Components/EffectShortcuts.cpp

@@ -7,9 +7,9 @@
 
 EffectShortcuts::EffectShortcuts() : Component()
 {
-	for (const auto &[effectId, effectData] : g_EnabledEffects)
-		if (effectData.ShortcutKeycode > 0 && !effectData.IsHidden())
-			m_AvailableShortcuts[effectData.ShortcutKeycode].push_back(effectId);
+	for (const auto &effectData : GetFilteredEnabledEffects())
+		if (effectData->ShortcutKeycode > 0 && !effectData->IsHidden())
+			m_AvailableShortcuts[effectData->ShortcutKeycode].push_back(effectData->Id);
 }
 
 void EffectShortcuts::OnRun()

ChaosMod/Components/Voting.cpp

@@ -304,8 +304,10 @@ void Voting::OnRun()
 
 bool Voting::Init()
 {
-	if (std::count_if(g_EnabledEffects.begin(), g_EnabledEffects.end(),
-	                  [](const auto &pair) { return !pair.second.IsExcludedFromVoting(); })
+	const auto enabledEffects = GetFilteredEnabledEffects();
+
+	if (std::count_if(enabledEffects.begin(), enabledEffects.end(),
+	                  [](const auto &effectData) { return !effectData->IsExcludedFromVoting(); })
 	    < 3)
 	{
 		ErrorOutWithMsg("You need at least 3 enabled effects (which are not excluded from voting) to enable voting."