fix(morphsummon.cpp): mage condition, closes #19 (#22)

* fix(morphsummon.cpp): mage condition, closes #19

* ci: do not run build twice for PRs

Author: Helias

.github/workflows/core-build.yml

@@ -1,6 +1,8 @@
 name: core-build
 on:
   push:
+    branches:
+      - 'master'
   pull_request:
   workflow_dispatch:
 

src/morphsummon.cpp

@@ -258,13 +258,15 @@ class MorphSummonCreatureScript : public CreatureScript
         bool sorry = false;
 
         // Mage Pet (minion)
-        if (Minion* minion = player->GetFirstMinion())
-        {
-            if (minion->GetUInt32Value(UNIT_CREATED_BY_SPELL) == SUMMON_WATER_ELEMENTAL)
+        if (player->getClass() == CLASS_MAGE) {
+            if (Minion* minion = player->GetFirstMinion())
             {
-                if (!mage_water_elemental.empty())
+                if (minion->GetUInt32Value(UNIT_CREATED_BY_SPELL) == SUMMON_WATER_ELEMENTAL)
                 {
-                    AddGossipItemFor(player, MORPH_GOSSIP_MENU_HELLO, MORPH_GOSSIP_OPTION_POLYMORPH, GOSSIP_SENDER_MAIN, MORPH_PAGE_START_MAGE_WATER_ELEMENTAL);
+                    if (!mage_water_elemental.empty())
+                    {
+                        AddGossipItemFor(player, MORPH_GOSSIP_MENU_HELLO, MORPH_GOSSIP_OPTION_POLYMORPH, GOSSIP_SENDER_MAIN, MORPH_PAGE_START_MAGE_WATER_ELEMENTAL);
+                    }
                 }
             }
         }