Misc/Util: Add a template function for returning a random element from any container

1 files changed, 1 insertions, 4 deletions

diff --git a/src/server/game/Entities/GameObject/GameObject.cpp b/src/server/game/Entities/GameObject/GameObject.cpp
index f08a7b2bc20..c0bf5865f28 100755
--- a/src/server/game/Entities/GameObject/GameObject.cpp
+++ b/src/server/game/Entities/GameObject/GameObject.cpp
@@ -1400,10 +1400,7 @@ void GameObject::Use(Unit* user)
                 {
                     for (int i = 0; i < info->summoningRitual.casterTargetSpellTargets; i++)
                     {
-                        std::set<uint32>::const_iterator itr = m_unique_users.begin();
-                        std::advance(itr, rand() % m_unique_users.size());
-
-                        if (Unit* target = Unit::GetUnit(*this, uint64(*itr)))
+                        if (Unit* target = Unit::GetUnit(*this, (uint64)SelectRandomContainerElement(m_unique_users)))
                             spellCaster->CastSpell(target, info->summoningRitual.casterTargetSpell, true);
                     }
                 }