[11838] Small code cleanup in Player::UpdateWeaponSkill

Thanks to stfx for providing the patch.

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11839] Add support to send custom gameobject anim for a few traps

The way to filter by display id is rather ugly, but no better solution is in sight. This should be good enough until better (likely model-data) support can be implemented.
Also it must be changed if more Traps with this behaviour are found.

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11840] Fix script command modify_npc_flag

Thanks to faq for pointing.
I checked both udb and ytdb, and they don't use this command yet;
If however you use it already check your scripts. (The toggle/add/remove flag must be set in `data_flags`) Sorry for the inconvenience.
Alos added a forgotten break.

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11841] Use difficulty selection also for player cast spells, example 65686

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11842] Add spell difficulty selection for creature*_addon auras

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11843] More improvements to LoadCreatureAddon, Thanks to Shauren and Vladimir for pointing
[11844] Implement spells 45976, 46177, 45989 for M'uru encounter

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11845] Implement ScriptEffects for spells 45141, 45151, 45206 for Brutallus

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
Update src/game/SpellEffects.cpp
[11846] Changes to default target selection

* Add a new function: Unit::IsSecondChoiceTarget to evaluate if a target will be selected
* Cleanup a bit related code
* SelectHostileTarget
 - Only call AI()->AttackStart for new targets
 - Remove exception for top-most taunter
* Fix a few minor bugs related to selectNextVictim

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11847] Fix teleport spells with TargetType (TARGET_TABLE_X_Y_Z_COORDINATES, NO_TARGET)

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
Updated SMSG_CAST_FAILED with some new information.
[11848] Change use of ScriptLibrary OnGOUse return value in GO::Use

Now mainly used to allow or prevent DB scripts use
Also some related code cleaned and reordered

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11849] Add missing pointer check for spell 45989.

Thanks to stfx for pointing

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11849] Implement Condition CONDITION_SKILL_BELOW to be able to check if a player has a skill lower than a given value

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
check unitTarget pointer
Update src/game/SpellEffects.cpp
[11850] Add additional flag SELECT_FLAG_NOT_IN_MELEE_RANGE

this allows to select targets that are not in melee range with SelectAttackingTarget

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11851] Implement spell effects for Kalecgos

Signed-off-by: Schmoozerd <schmoozerd@scriptdev2.com>
[11852] Add support for db scripts assigned to gossip menus