[feature] Updated pull feature now some way done (checks only, attack functionality removed). Improved CombatRestore but may not be fully functioning at the moment (CombatOrder-wise). Various bits 'n pieces. Several check functions used in pull but useful in other ways in the future.
Signed-off-by: Cala Haryon <calaftp@free.fr>
Fix to prevent merge conflicts, when sql changes occur &
 provide a more relevant db error message, should the versions
 not match.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] Merge primorder and secorder into combat_order for db as well as code. Add safety check so only a single primary order can be active. Streamline various combat order code. Merge resist types into combat_order.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] tiny commented out improvement to [still non-functioning] pulling feature.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] One step/check closer to a working 'pull' function.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[fix] small fix to druid CanPull check function.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[fix] playerbot_saved_data has been adjusted to new format.

Signed-off-by: Cala Haryon <calaftp@free.fr>
Fix for combat orders

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] One step closer to a working pull function.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[fix] Fix all PBAI Orders checks (no == but & checks instead because the states are combinable). Fix several pull bugs. Slight amount of extra pull code.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[fix] Extra pull code. Missing it was causing compile error - oops.

Signed-off-by: Cala Haryon <calaftp@free.fr>
 [tweak] More pull code, slight reorganization thereof. Extended functions include 'pull test' and 'pull ready', see 'help pull'. Those two functions are actually fully functional, only 'pull' is not.

Signed-off-by: Cala Haryon <calaftp@free.fr>
Renamed CombatDelayRestore to BotDataRestore

Signed-off-by: Cala Haryon <calaftp@free.fr>
 [fix] Faulty PBAI::CanPull() check. 'pull test' now also tells you when succesful.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] More pull code, pull itself is still not functional.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[fix] CanPull() checks are now class-specific rather than always failing. 'pull' command will now actually initiate combat with non-combatees starting 2 seconds behind. This makes the 'pull' command mildly functional though certainly not working as intended.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[feature] Fully functional pull method. Could use some tweaks (tank not moving towards target unless required) and fixes (why will warriors not use gun/bow?) but despite all that, fully functional. Tank 'pulls', attacks by itself, rest joins in 2 seconds after tank does melee attacks - or once another group member gains aggro or after 10 seconds.

Signed-off-by: Cala Haryon <calaftp@free.fr>
 [fix] tanks are now properly setting TARGET_THREATEN mode - even if that flag's ignored everywhere.

Signed-off-by: Cala Haryon <calaftp@free.fr>
 [tweak] pull no longer causes tank to melee attack until target is in range. Various tweaks to GetCombatTarget and related code.

Signed-off-by: Cala Haryon <calaftp@free.fr>
[tweak] warrior pull melee tweak. Probably moot (pulling a non-hostile in melee range? What are the chances?) but better safe than sorry.

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