Extracts CastSpell() out of priest, mage, warlock and into PlayerbotClassAI::CastSpellWand(). Contains a few other minor tweaks. Rest of the code changes are because CastSpellWand + CastSpell + DoNextCombatManeuver now return a bool. Do not use this return value (yet), only mage, priest and warlock properly return this; All other classes always return false (as though DoNextCombatManeuver() has failed).

Signed-off-by: cala <calaftp@free.fr>
Fix: m_ai is there but never used. Fixed for mage. Rest to follow.

Signed-off-by: cala <calaftp@free.fr>
[Tweak] Minor druid tweaks.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Compile fix to previous commit.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Introduce usage of m_ai to all classes. Normalize checks for m_ai and m_bot throughout each class.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Introduce usage of m_ai to PlayerbotClassAI. Normalize checks for m_ai and m_bot as well.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Use m_ai in Druid, plus a few other minor tweaks.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Compile fix for earlier commit.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Fix and much clearer code for PlayerbotAIDruid::CheckForm().

Signed-off-by: cala <calaftp@free.fr>
[Tweak] Much improved shapeshifting management. Much improved rotation management for feral users. Also generally cleaner looking code.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Fixed a spec check that would default all Feral specs to Balance.
Also includes some tweaks for cleaner code. Plus, Druid will cast damage
spells now.

Signed-off-by: cala <calaftp@free.fr>
Compile fix for enum labels

Signed-off-by: cala <calaftp@free.fr>
[Fix] PlayerbotClassAI: Longer combat delay properly deals with multiple short fights. Renamed the two pertinent variables to Hungarian notation (I can only see it so often before the uncontrollable urge to make it conform). Each class's AI: squelched compiler warnings by closing each DoNextCombatManeuver by a return statement. Also, wand classes should no longer mention they cannot find a spell due to being out of mana (although if they're low on mana they may not say it even if it IS warranted).

Signed-off-by: cala <calaftp@free.fr>
[Fix] Wrong logical check... Fix is a generous term - still broken in general but one step closer.

Signed-off-by: cala <calaftp@free.fr>
[Fix] Bots will once again fight. ... But now combat delay is ignored (still, an improvement). Bots can still get stuck, and casters still superspam spells. New record, my warlock got off 10+ shadowbolts in 1 second :)

Signed-off-by: cala <calaftp@free.fr>
[Tweak] Small clarification.

Signed-off-by: cala <calaftp@free.fr>
A few tweaks to improve game play
 o Remove glitch: When bot(s) 'attack' they appear to doubleback, before continuing to target
 o Remove glitch: Where bot(s) would slow up as they loot targets after combat
 o Improve: Selectivity of closely spaced npc by bot(s)

Signed-off-by: cala <calaftp@free.fr>
Clean code of PlayerbotAI::UpdateAI()

Signed-off-by: cala <calaftp@free.fr>
[Fix] FirstCombatManeuver was followed directly by NextCombatManeuver. Fix also comes with more detailed return values by FirstCombatManeuver and NextCombatManeuver. Also includes slight adjustment to Rogue's combat rotation (concerning combo moves).

Signed-off-by: cala <calaftp@free.fr>
[Tweak] Uncommented FirstCombatManeuver for Warrior bots, restructured the code, checked return values. Not sure what the issue was before but should work now (and much clearer to boot).

Signed-off-by: cala <calaftp@free.fr>