Core/Spells: Fix Rogue T10 4P bonus
Closes #4153
Core/SAI: Additions to d0bde02b21241773c908c18ad

Signed-off-by: Nay <dnpd.dd@gmail.com>
Core/Vehicle: Fix spell focus check for vehicle caster
Fix non-pch compile
Core/Spells:
- corrected structure of SMSG_PET_CAST_FAILED (fixes highlighted action buttons)
- simplified spell focus check
DB/Loot: Fix some horrible loot definitions for Herbalism, reworked ALL Herbs
DB/Loot: while at it also did the mining veins
SQL: Fix import error
Core/Spells: implemented SPELL_ATTR0_CU_TRIGGERED_BY_CASTER to correct some triggered spells, which will wrongly triggered by target instead of caster

Closes #1972
DB/Gameobject: Spawn or respawn missing or bad spawned gameobjects "Coldwind Tree" / "Poster Knife"
DB/commit: the mining is also useable for 434, thus removed the _335 tag
Core/SmartAI: Implement SMART_EVENT_FRIENDLY_HEALTH_PCT.

 On current clean TDB, there are a total of 106 creatures using
   SMART_EVENT_FRIENDLY_HEALTH (non-PCT) which has a flat first parameter, but
   only 8 of these use the event correctly (and therefore only 8 of them
   actually work).

Closes #10520
Core/SmartAI: Implement SMART_TARGET_CLOSEST_FRIENDLY.

Parameter is distance to search for.

Closes #10521
Lumber Hack (12050)

Fixes #4560
Revert "Lumber Hack (12050)"

This reverts commit 40ec7f7979cd65fccc6fe74c056688da4c4d47ce.
[DB/SAI]: Lumber Hack (12050)
SQL: Add missing newline in 5890f95f7cd7a885b667fe9b5da2dceea22b9b81
Core/Auras: Allow preventing whole aura proc default handler from script hooks, just how it is possible for separate effects
Core/Spells: Fixed cast time mods using two charges on every cast
Core/Spells: Refactored SpellInfo::CalcCastTime() to prevent accidental mod charge losses caused by its incorrect usage