Wraper functions for 8.0.1 spell/aura changes

[Edik]

In 7.3 you won't need any caching. In 8.x is quering Aura by name completely removed. So if you have mutiple checks in single line I guess caching is worth to try. If you do by aura index from 1 to 40 old fashioned way you won't need to cache. Just imagine if you have this in your old code.

function WWM:isEating() -- am I bussy by feeding self?
  return UnitAura(ID_PLAYER, BUFF_FOOD) or UnitAura(ID_PLAYER, BUFF_DRINK) or UnitAura(ID_PLAYER, BUFF_REFRESHMENT) or UnitAura(ID_PLAYER, BUFF_DRINK_AND_FOOD)
end

I had my wrappers a small glitch I did fix it original post.

[boomboo]

Originally Posted by Edik In 7.3 you won't need any caching. In 8.x is quering Aura by name completely removed. So if you have mutiple checks in single line I guess caching is worth to try. If you do by aura index from 1 to 40 old fashioned way you won't need to cache. Just imagine if you have this in your old code.

I know, I was talking from the 8.x perspective, and the comparrisons I ran were with single spell querying. As I mentioned though the overhead of all the data being copied and cleared wasn't worth it. You could run the tests yorself but I doubt you'll get different results.

Also UnitAura doesn't support querying by spellid in either 7.3 or 8.x, the second argument if numeric is just an index.