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
Scripted/BaradinHold: Implemented Occu'thar
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
Merge branch 'master' of github.com:TrinityCore/TrinityCore into 4.3.4

Conflicts:
	src/server/game/Entities/Unit/Unit.cpp
	src/server/game/Entities/Unit/Unit.h
	src/server/game/Handlers/PetHandler.cpp
	src/server/game/Spells/Spell.cpp
	src/server/game/Spells/Spell.h
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
Scripted/BaradinHold: Add missing door for Alizabal
Scriped/BaradinHold: Small cleanup in 595fcdee70d5ef54986d1450e24874e08d5cbdfa
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
Misc: Random changes here and there
Merge branch 'master' of github.com:TrinityCore/TrinityCore into 4.3.4

Conflicts:
	src/server/game/Spells/SpellInfo.cpp
Core/Spells: Fix priest Shadowfiend
Lumber Hack (12050)

Fixes #4560