pa/eqemu-server
2
0
Fork 0
mirror of https://github.com/EQEmu/Server.git synced 2026-03-10 10:12:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Use StopCasting instead of Interrupt on SpellFinished failure

Browse Source 
Most of these failures already show a message, live doesn't double
up on failure message + interrupt message, so lets not

There are a few logic error returns, but those really shouldn't happen
This commit is contained in:
Michael Cook (mackal) 2016-08-29 13:27:42 -04:00
parent 95ea61114a
commit ea1ae1a0a4
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
zone/spells.cpp
View File

@ -1310,7 +1310,9 @@ void Mob::CastedSpellFinished(uint16 spell_id, uint32 target_id, CastingSlot slo
if( !SpellFinished(spell_id, spell_target, slot, mana_used, inventory_slot, resist_adjust) ) if( !SpellFinished(spell_id, spell_target, slot, mana_used, inventory_slot, resist_adjust) )
{ {
Log.Out(Logs::Detail, Logs::Spells, "Casting of %d canceled: SpellFinished returned false.", spell_id); Log.Out(Logs::Detail, Logs::Spells, "Casting of %d canceled: SpellFinished returned false.", spell_id);
InterruptSpell(); // most of the cases we return false have a message already or are logic errors that shouldn't happen
// if there are issues I guess we can do something else, but this should work
StopCasting();
return; return;
} }