Core/Conditions: Implement condition to check if player is in group (TrinityCore#31858)

Co-authored-by: Shauren <shauren.trinity@gmail.com>

Author: offl

sql/updates/world/3.3.5/2026_07_16_00_world.sql

@@ -0,0 +1,27 @@
+--
+DELETE FROM `spell_target_position` WHERE `ID` IN (12885,13142,27694);
+INSERT INTO `spell_target_position` (`ID`, `EffectIndex`, `MapID`, `PositionX`, `PositionY`, `PositionZ`, `Orientation`, `VerifiedBuild`) VALUES
+(12885,0,0,-11235.3,-2832.14,157.925,4.71,0),
+(13142,0,0,-11235.3,-2832.14,157.925,4.71,0),
+(27694,0,0,-11235.3,-2832.14,157.925,4.71,0);
+
+DELETE FROM `gossip_menu_option` WHERE `MenuID` = 1541 AND `OptionID` = 1;
+INSERT INTO `gossip_menu_option` (`MenuID`,`OptionID`,`OptionIcon`,`OptionText`,`OptionBroadcastTextID`,`OptionType`,`OptionNpcFlag`,`ActionMenuID`,`ActionPoiID`,`BoxCoded`,`BoxMoney`,`BoxText`,`BoxBroadcastTextID`,`VerifiedBuild`) VALUES
+(1541,1,0,"We wish to face the Defiler.",11869,1,1,0,0,0,0,"",0,0);
+
+DELETE FROM `smart_scripts` WHERE `entryorguid` = 8816 AND `source_type` = 0;
+INSERT INTO `smart_scripts` (`entryorguid`,`source_type`,`id`,`link`,`event_type`,`event_phase_mask`,`event_chance`,`event_flags`,`event_param1`,`event_param2`,`event_param3`,`event_param4`,`event_param5`,`action_type`,`action_param1`,`action_param2`,`action_param3`,`action_param4`,`action_param5`,`action_param6`,`target_type`,`target_param1`,`target_param2`,`target_param3`,`target_param4`,`target_x`,`target_y`,`target_z`,`target_o`,`comment`) VALUES
+(8816,0,0,0,62,0,100,0,1541,0,0,0,0,11,12885,0,0,0,0,0,7,0,0,0,0,0,0,0,0,"Deathly Usher - On Gossip Option 0 Selected - Cast 'Teleport to Razelikh'"),
+(8816,0,1,0,62,0,100,0,1541,1,0,0,0,11,27686,0,0,0,0,0,7,0,0,0,0,0,0,0,0,"Deathly Usher - On Gossip Option 1 Selected - Cast 'Teleport to Razelikh (GROUP)'");
+
+DELETE FROM `conditions` WHERE `SourceTypeOrReferenceId` = 15 AND `SourceGroup` = 1541;
+INSERT INTO `conditions` (`SourceTypeOrReferenceId`,`SourceGroup`,`SourceEntry`,`SourceId`,`ElseGroup`,`ConditionTypeOrReference`,`ConditionTarget`,`ConditionValue1`,`ConditionValue2`,`ConditionValue3`,`ConditionStringValue1`,`NegativeCondition`,`ErrorType`,`ErrorTextId`,`ScriptName`,`Comment`) VALUES
+(15,1541,0,0,0,47,0,3628,10,0,'',0,0,0,'',"Group 0: Show Gossip Option 0 if player has taken quest 'You Are Rakh'likh, Demon'"),
+(15,1541,0,0,0,2,0,10757,1,0,'',0,0,0,'',"Group 0: Show Gossip Option 0 if player has item 'Ward of the Defiler'"),
+(15,1541,0,0,0,60,0,0,0,0,'',0,0,0,'',"Group 0: Show Gossip Option 0 if player is not in group"),
+
+(15,1541,1,0,0,47,0,3628,10,0,'',0,0,0,'',"Group 0: Show Gossip Option 1 if player has taken quest 'You Are Rakh'likh, Demon'"),
+(15,1541,1,0,0,2,0,10757,1,0,'',0,0,0,'',"Group 0: Show Gossip Option 1 if player has item 'Ward of the Defiler'"),
+(15,1541,1,0,0,60,0,1,0,0,'',0,0,0,'',"Group 0: Show Gossip Option 1 if player is in group");
+
+UPDATE `spell_script_names` SET `ScriptName` = 'spell_blasted_lands_teleport_to_razelikh_group' WHERE `ScriptName` = 'spell_razelikh_teleport_group';

src/server/game/Conditions/ConditionMgr.cpp

@@ -21,6 +21,7 @@
 #include "DatabaseEnv.h"
 #include "GameEventMgr.h"
 #include "GameObject.h"
+#include "Group.h"
 #include "InstanceScript.h"
 #include "Log.h"
 #include "LootMgr.h"
@@ -127,8 +128,28 @@ ConditionMgr::ConditionTypeInfo const ConditionMgr::StaticConditionTypeData[COND
     { "Private Object",           false, false, false, false },
     { "String ID",                false, false, false,  true },
     { "Label",                    false, false, false, false },
+    { "Group status",              true, false, false, false }
 };
 
+static bool MeetsGroupStatusCondition(Player const* player, GroupStatusCondition status)
+{
+    Group const* group = player->GetGroup();
+    switch (status)
+    {
+        case GroupStatusCondition::NotInGroup:
+            return group == nullptr;
+        case GroupStatusCondition::InGroup:
+            return group != nullptr;
+        case GroupStatusCondition::InGroupButNotInRaid:
+            return group && !group->isRaidGroup();
+        case GroupStatusCondition::InRaid:
+            return group && group->isRaidGroup();
+        case GroupStatusCondition::NotInGroupOrNotInRaid:
+            return !group || !group->isRaidGroup();
+    }
+    return false;
+}
+
 // Checks if object meets the condition
 // Can have CONDITION_SOURCE_TYPE_NONE && !mReferenceId if called from a special event (ie: SmartAI)
 bool Condition::Meets(ConditionSourceInfo& sourceInfo) const
@@ -548,6 +569,12 @@ bool Condition::Meets(ConditionSourceInfo& sourceInfo) const
                 condMeets = go->HasStringId(ConditionStringValue1);
             break;
         }
+        case CONDITION_GROUP_STATUS:
+        {
+            if (Player const* player = object->ToPlayer())
+                condMeets = MeetsGroupStatusCondition(player, GroupStatusCondition(ConditionValue1));
+            break;
+        }
         default:
             condMeets = false;
             break;
@@ -750,6 +777,9 @@ uint32 Condition::GetSearcherTypeMaskForCondition() const
         case CONDITION_STRING_ID:
             mask |= GRID_MAP_TYPE_MASK_CREATURE | GRID_MAP_TYPE_MASK_GAMEOBJECT;
             break;
+        case CONDITION_GROUP_STATUS:
+            mask |= GRID_MAP_TYPE_MASK_PLAYER;
+            break;
         default:
             ABORT_MSG("Condition::GetSearcherTypeMaskForCondition - missing condition handling!");
             break;
@@ -2401,6 +2431,13 @@ bool ConditionMgr::isConditionTypeValid(Condition* cond) const
                 return false;
             }
             break;
+        case CONDITION_GROUP_STATUS:
+            if (cond->ConditionValue1 > uint32(GroupStatusCondition::NotInGroupOrNotInRaid))
+            {
+                TC_LOG_ERROR("sql.sql", "{} has non invalid group status condition value1 ({}), skipped.", cond->ToString(true), cond->ConditionValue1);
+                return false;
+            }
+            break;
         case CONDITION_IN_WATER:
         case CONDITION_CHARMED:
         case CONDITION_TAXI:

src/server/game/Conditions/ConditionMgr.h

@@ -95,6 +95,7 @@ enum ConditionTypes
     CONDITION_PRIVATE_OBJECT           = 57,                   // 0                      0              0                  true if entity is private object
     CONDITION_STRING_ID                = 58,
     CONDITION_LABEL                    = 59,                   //                                                          only for master branch
+    CONDITION_GROUP_STATUS             = 60,                   // GroupStatus            0              0                  true if player group status is (0 = not in group, 1 = in group, 2 = in group but not in raid, 3 = in raid group, 4 = not in group or not in raid)
     CONDITION_MAX
 };
 
@@ -183,6 +184,15 @@ enum InstanceInfo
     INSTANCE_INFO_DATA64
 };
 
+enum class GroupStatusCondition : uint32
+{
+    NotInGroup              = 0,
+    InGroup                 = 1,
+    InGroupButNotInRaid     = 2,
+    InRaid                  = 3,
+    NotInGroupOrNotInRaid   = 4
+};
+
 enum MaxConditionTargets
 {
     MAX_CONDITION_TARGETS = 3

src/server/scripts/EasternKingdoms/zone_blasted_lands.cpp

@@ -17,52 +17,39 @@
 
 #include "ScriptMgr.h"
 #include "SpellScript.h"
-#include "Player.h"
-#include "Group.h"
+#include "Unit.h"
 
 /*######
 ## Quest 3628: You Are Rakh'likh, Demon
 ######*/
 
-enum TeleportToRazelikh
+enum YouAreRakhlikhDemon
 {
-    SPELL_TELEPORT_SINGLE               = 12885,
-    SPELL_TELEPORT_SINGLE_IN_GROUP      = 13142
+    SPELL_TELEPORT_TO_RAZELIKH_GROUP    = 27694
 };
 
 // 27686 - Teleport to Razelikh (GROUP)
-class spell_razelikh_teleport_group : public SpellScript
+class spell_blasted_lands_teleport_to_razelikh_group : public SpellScript
 {
-    PrepareSpellScript(spell_razelikh_teleport_group);
+    PrepareSpellScript(spell_blasted_lands_teleport_to_razelikh_group);
 
-    bool Validate(SpellInfo const* /*spell*/) override
+    bool Validate(SpellInfo const* /*spellInfo*/) override
     {
-        return ValidateSpellInfo({ SPELL_TELEPORT_SINGLE, SPELL_TELEPORT_SINGLE_IN_GROUP });
+        return ValidateSpellInfo({ SPELL_TELEPORT_TO_RAZELIKH_GROUP });
     }
 
-    void HandleScriptEffect(SpellEffIndex /* effIndex */)
+    void HandleScript(SpellEffIndex /*effIndex*/)
     {
-        if (Player* player = GetHitPlayer())
-        {
-            if (Group* group = player->GetGroup())
-            {
-                for (GroupReference* itr = group->GetFirstMember(); itr != nullptr; itr = itr->next())
-                    if (Player* member = itr->GetSource())
-                        if (member->IsWithinDistInMap(player, 20.0f) && !member->isDead())
-                            member->CastSpell(member, SPELL_TELEPORT_SINGLE_IN_GROUP, true);
-            }
-            else
-                player->CastSpell(player, SPELL_TELEPORT_SINGLE, true);
-        }
+        GetHitUnit()->CastSpell(nullptr, SPELL_TELEPORT_TO_RAZELIKH_GROUP, true);
     }
 
     void Register() override
     {
-        OnEffectHitTarget += SpellEffectFn(spell_razelikh_teleport_group::HandleScriptEffect, EFFECT_0, SPELL_EFFECT_SCRIPT_EFFECT);
+        OnEffectHitTarget += SpellEffectFn(spell_blasted_lands_teleport_to_razelikh_group::HandleScript, EFFECT_0, SPELL_EFFECT_SCRIPT_EFFECT);
     }
 };
 
 void AddSC_blasted_lands()
 {
-    RegisterSpellScript(spell_razelikh_teleport_group);
+    RegisterSpellScript(spell_blasted_lands_teleport_to_razelikh_group);
 }