cmangos/mangos-tbc — pull request #19 by cala (2013-05-10 12:50:07+00:00)
Title: Fix take ammo for most ranged spells

Fix spells like Arcane Shot not taking ammo while they should

This commit []() is one of those pending backports from old mangos-zero concatenated in: https://github.com/cmangos/issues/wiki/classic_backporting-todo-zero-commits

stfx adviced to first apply to TBC, then backport to classic

I tested it and this is my report:
> cala: tested take ammo and found issue on current revision: ammo are not taken for all spells (ex: ‘arcane shot’ does not take ammo but ‘serpent sting’ does). However, if no ammo in inventory no ranged spell can be casted, even those like ‘arcane shot’ that do not take ammo (though probably should take). I’ll try to test this commit.
stfx: was this behavior ever changed? it should be tested/fixed on tbc also if needed
cala (march 13-13): this fix does allow all spells to take ammo. It should be applied to TBC firsthand, then backported to classic.