*Remove not needed dummy effect handler for spell 28006 - by azazel_kon

*Add CanCompleteQuest in Spell::EffectQuestComplete - thanks to azazel_kon for research.
*Fixes issue #3011

--HG--
branch : trunk

1 files changed, 1 insertions, 8 deletions

diff --git a/src/server/game/Spells/SpellEffects.cpp b/src/server/game/Spells/SpellEffects.cpp
index 1ed11b22e0a..b1f30e9ba1b 100644
--- a/src/server/game/Spells/SpellEffects.cpp
+++ b/src/server/game/Spells/SpellEffects.cpp
@@ -1109,13 +1109,6 @@ void Spell::EffectDummy(uint32 i)
                 case 26074:                                 // Holiday Cheer
                     // implemented at client side
                     return;
-                case 28006:                                 // Arcane Cloaking
-                {
-                    if (unitTarget && unitTarget->GetTypeId() == TYPEID_PLAYER)
-                        // Naxxramas Entry Flag Effect DND
-                        m_caster->CastSpell(unitTarget, 29294, true);
-                    return;
-                }
                 // Polarity Shift
                 case 28089:
                     if (unitTarget)
@@ -7042,7 +7035,7 @@ void Spell::EffectQuestComplete(uint32 i)
         uint16 log_slot = pPlayer->FindQuestSlot(quest_id);
         if (log_slot < MAX_QUEST_LOG_SIZE)
             pPlayer->AreaExploredOrEventHappens(quest_id);
-        else
+        else if (CanCompleteQuest(quest_id))
             pPlayer->CompleteQuest(quest_id);   // quest not in log - for internal use
     }
 }