Quest reward spells are cast incorrectly...

This is a db issue, there needs to be a quest end script for this, please tell me the ques t id to fix it

yeah...it's not db error,it's core error- -! i fixed it.

src/game/Object/Player.cpp | 22 ++++++++++++++++++++--
1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/src/game/Object/Player.cpp b/src/game/Object/Player.cpp
index 0915f4f..127a4e0 100644
--- a/src/game/Object/Player.cpp
+++ b/src/game/Object/Player.cpp
@@ -12322,9 +12322,27 @@ void Player::RewardQuest(Quest const* pQuest, uint32 reward, Object* questGiver,

// cast spells after mark quest complete (some spells have quest completed state reqyurements in spell_area data)
if (pQuest->GetRewSpellCast() > 0)
- { CastSpell(this, pQuest->GetRewSpellCast(), true); }
+ {
+ SpellInfo const* spellInfo = sSpellStore.LookupEntry(pQuest->GetRewSpellCast());
+ if (questGiver->isType(TYPEMASK_UNIT) && !spellInfo->HasEffect(SPELL_EFFECT_LEARN_SPELL) && !spellInfo->HasEffect(SPELL_EFFECT_CREATE_ITEM))
+ {
+ if (Creature* creature = GetMap()->GetCreature(questGiver->GetGUID()))
+ creature->CastSpell(this, pQuest->GetRewSpellCast(), true);
+ }
+ else
+ CastSpell(this, pQuest->GetRewSpellCast(), true);
+ }
else if (pQuest->GetRewSpell() > 0)
- { CastSpell(this, pQuest->GetRewSpell(), true); }
+ {
+ SpellInfo const* spellInfo = sSpellStore.LookupEntry(pQuest->GetRewSpell());
+ if (questGiver->isType(TYPEMASK_UNIT) && spellInfo->HasEffect(SPELL_EFFECT_LEARN_SPELL) && !spellInfo->HasEffect(SPELL_EFFECT_CREATE_ITEM))
+ {
+ if (Creature* creature = GetMap()->GetCreature(questGiver->GetGUID()))
+ creature->CastSpell(this, pQuest->GetRewSpell(), true);
+ }
+ else
+ CastSpell(this, pQuest->GetRewSpell(), true);
+ }

// remove auras from spells with quest reward state limitations
// Some spells applied at quest reward

Unfortunately, this copypaste from TC will not compile in Zero due to missing SpellInfo class.

The problem of the casting mode for such spells, either player->player or questender->player, looks permanent :) The correct solution seems to use both modes depending on conditions. The above patch provides an incomplete list of the conditions.

In general, the questender->player mode seems to be the reasonable one, except the spells which cannot be casted in this mode. Most of later spells have TARGET_SELF=1 (we don't need usually the questender to cast the reward spell on itself, do we?). After a brief look on the spells actually used for RewSpell, we've to add the spells either with TARGET_NONE=0 (the core interprets it as TARGET_SELF) or with SPELL_EFFECT_LEARN_SPELL=36, SPELL_EFFECT_TRADE_SKILL=47. The difference between these two solutions on the actual DB arises just for dummy auras 23356,23357. Checking if the core allows .cast back the auras from mob to player will solve the dilemma: if yes, then 2nd way (check spell effects); else 1st way (check TARGET_NONE=0 for non-empty spell effects).

Again, due to missing SpellInfo class, a special method has to be created to check the mentioned conditions correctly basing on SpellEntry entity.

OK , it's a patch from TC.BUT it's not hard to buid spellinfo class.So I did it ,but not all compelete:)
I think it necessary to write spellinfo class for mangoszero.we could do it in develop21