[Compiles] Mangos + YTDB + Scriptdev2 + Changelog

[TheUndyingPharaoh]

Awesome
+Rep

[Dj Aligator]

Mangos 10611 - 10636

Made some values const
Revert "[10611] Made some values const"

Reason: It makes no sense to make a local variable const.

This reverts commit 49e3184d69056738701bc15d9b7f982bf11d7bbc.
A bit of cleanup in ByteBuffer::hexlike
Replace ACE_ASSERT with MANGOS_ASSERT
Add script effect of spell 66741
Simplified trade distance checks, GROUP_UPDATE_FLAG_POSITION flag handling moved out of Map

note: need avoid Map::PlayerRelocation calls now and use Player::SetPosition instead
Fixed array access in SCRIPT_COMMAND_TALK loading check.
Properly update objects usable state at GM-mode on/off.
Clarify premade groups mode config option.

Also prevent mark group as premade with 0 time wait.
It's any way useless becase at next tick group moved to normal team mode.
Fixed MSG_MOVE_HEARTBEAT structure.

Thx Wojta for pointing.
Add new field RequiredClasses for quest_template

* SkillOrClass is converted to RequiredSkill (and then field can contain skill id only)
* Field ZoneOrSort has no longer a function in quest requirement, and RequiredClasses must be used instead where class limits are expected.

To restrict a quest to one class or more, use bitmask of class in RequiredClasses. RequiredSkill works like before.
Set respawn delay for BG creatures before set dead state.

This prevent save to DB wrong respawn time and unexpected in wrong event state spawn BG creatures.
Make .respawn work correctly with BG event spawns
Fix quests w/QuestFlags QUEST_FLAGS_AUTO_REWARDED
Implement SPELL_EFFECT_CLEAR_QUEST (139)
Fix bad bandate check.
Fix logic in FillRaidOrPartyManaPriorityTargets and FillRaidOrPartyHealthPriorityTargets
Use ObjectGuid for player's selection and unit's target
SQL update for prev. commit
Drop function GuidHigh2TypeId

Added ObjectGuid version Aura::GetCasterGuid()

But for prevent big problen with backporting AuraHolder
to mangos-0.12 in most cases still used old uint64 version
Aura::GetCasterGUID.

Recommended use new version where possible in clean way
as done for some cases in commit already.
Convert Set(Get)ChannelObjectGuid to ObjectGuid way.

Also fix typo in debug output for prev. commit.
Small code style cleanup in quest related functions
Small comment style fix in mangosd.conf
Correct behavior for completion of quest objectives.

Rename related function to SendQuestCompleteEvent for clarification of when it's expected to be used.
Include non-percent mana cost in EffectDestroyAllTotems mana return.

Also small speedup for no-return spells.
Convert many opcode handlers to ObjectGuid use.

Also re-enabled anticheating check for bank window open
base at .bank command access rights.
Ignore another map target for text emote.
Implement server side check combo points req. for spell cast.

Thanks to nos4r2zod for problem research.

YTDB 347

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus
+ new templates from WDB
+ update table "*_template" by WDB
+ data from sniffs (templates, spawn)
+ processed 220 caches (3.3.5a)

[Dj Aligator]

Mangos 10637 - 10658

Better totem summon cases selection.

But added check still not allways work. Unclear better way selections for summon prop id 61
More appropriate for warrior spell fail result for combo point check fail.
Making dummy effect of spell 54530 be positive effect.
Implement instance guids with proper HIGHGUID_INSTANCE use.
Fixed use item dynamic/item proto flags.

* List its independelty. Each from this 2 fields have own flags. Cleanup lists.
* Not copy proto flags to item flags update field. This fix heroic item versions
  show in client in result mixed use ITEM_DYNFLAG_WRAPPED (0x8) with ITEM_FLAG_HEROIC (0x8)
* Update uses to proto case use for some cases where before wrongly has been used item dyn.flags
* For ITEM_DYNFLAG_UNLOCKED (0x4) set for item at unlock and check for prevent re-unlocking.
  This mostly for future case when will be implemented partly loting items support.
* For ITEM_FLAG_LOOTABLE (0x4) check loot absent or conflicting setting with containers/casted at use items.
  Report wrong cases art loading.
* Better check related loot tables content using ITEM_FLAG_PROSPECTABLE and ITEM_FLAG_MILLABLE
Implement spell 56826
Update the ACE part we use to 5.8.2
Remove wrong check preventing lootable itesm have at-use spell casts.

But note: like at use spells cast at loot open not implemnted yet.

Also fixed typo in ITEM_DYNFLAG_READABLE.
Use more make sense corpse expire check timer period.

Use 3 hours timer when bones really expired by Corpse::IsExpired code each 1 hr
just lol ;) So restore old 20 min time used long time ago.

Thanks to TGM for pointing to patch.
Preserve creature dynamic flags at UpdateEntry
Show die animation at totem destroy.

Or any creature type die with linked unsummon.

Original patch provided by Eggxp.
Typos in mangosd.conf.dist.in

Also thanks to Marik for additional typo fix and diff preparing
 Enchanted spell 8017 and ranks should apply to weapon by client auto selection
One more typo in mangosd.conf.dist.in
Fixed combat log being bugged by pets without name
Allow set pvp flags from creatures *_addon data
Implemented MSG_MOVE_KNOCK_BACK opcode

 * Player's knockback movement now smooth and doesn't looks like teleporting
 * Cleanup WorldSession::HandleMovementOpcodes, separated opcode specific and generic movement handlers code
 * Handle Feign Death aura interrupting into more appropriate place
Allow creature to offer same quest for more than one game event
Add support for monthly quests

Quest that can repeated each month are set by quest_template.SpecialFlags |0x04 flag
Quest are reset at midnight the first day of each month.
Note: for the time being, quest must also be set repeatable (SpecialFlags |0x01)
Rename quest function HasFlag to HasQuestFlag
Separate quest_template.QuestFlags from SpecialFlags

Create enum for SpecialFlags (database flags and internally computed)
Added related functions for specialFlags and update code accordingly.
Add enum as argument for quest special/flags functions

YTDB 348

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

[Dj Aligator]

Long time no updates


Mangos 10659 - 10737

Fixed mistake in [10653] that causes player to fall from the transport at teleporting to another map
Make quest for game event be independent of source

Create new table and convert existing data.
Simplify how game event quests are activated during event by adding generic function to set quest active/inactive.
Any quest in game_event_quest are disabled until event start (and deactivated once stopped)
Update VC10 filers for new ACE
Implement partly item looting

* Not deleted normal item loot at first loot window close.
* Save current item loot content to DB for later continue looting.
* Cleanup spell effect SPELL_EFFECT_OPEN_LOCK check code
* Cleanup Player::AutoStoreLoot calls
Fix totem animation at normal unsummon.
Rename arena_team_stats fields for better fit to stored values.
Allow use faction data for cases raceMask==0

Also some related code cleanup
Implement ACHIEVEMENT_CRITERIA_TYPE_HIGHEST_*_RATING
Timed Achievements, Fail-Support and Opcodes
Implement CONDITION_*ITEM_WITH_BANK conditions.

Similar CONDITION_*ITEM but including character bank content in counting.
Fixed mana amount for overheal case for talent 31785 and ranks.

Also thanks to Wojta for patch code improvements.
Restore build at real C++ compilers after recent timed achievements commit.
Convert some Unit owner/etc guids to ObjectGuid way.
Allow damage own pet in sanctuary.

Also mome repeating code to new function.
Include `item_loot` into pdumps
Show some SPELL_AURA_MOD_DODGE_PERCENT auras as debufs as expected.
Ignore BOA items reputation requirements at use.

Item reputation requirements checked at buy, and in quest case
related quests expected to have own reputation requirements
preventing rewarding items with not fit reputation requirements.
Fix repeating looting for milling/prospecting.
Send to creature/etc Update call real diff from last update and use it.

Now in case when creature/etc some tices not updates in result stay
in not active (no near players or active objects) cell some important
timers (corpse decay, summon timers, group loot expire, aura durations, etc) will
updates at real diff time from last prev. update call.

For some systems added exclude use real diff time because current limitations
like move generators. So its stay use last tick diff and considered freeze and
skip all time while creature in not active map part.
Revert "[10675] Ignore BOA items reputation requirements at use."

This reverts commit 4ce902e5d9c3cfc635c0eff1ba3552da5ecd0c5b.

Check done at client side so useless skip it at server side.
Proper fix expect show diff items in vendor list depndent from required reputation.
New table `npc_vendor_template` for comon item data for diff vendors.

In vendor list show own vedor items + template ites if any.
Vendor template item lists selected by id stored in creature_template.vendor_id
Update item buy code for new way vendor item lists work.
Added new Cell::Visit* functions to visit grids with center in (x,y) coords

* This fixes possible problems with Spell::FillAreaTargets(center of search should be (x,y) position, not spell caster's position)
* Cleanup Cell class, removed old and unused code
New table 'item_convert' for convertion items.

* Implment vendor at show replace BoA item with reputation requirement
  by identical item without rep. reqs for player that fit this requirements
  of original item.

* Table also can be used for store item convertion data form one team items
  to similar other team items. This case not impement but added function easy
  used for it.

* Note: strict single convert for item not created problem for both way use:
  item converted for same team or all, can't be original item for conversion to
  another team.
Revert "[10677] Send to creature/etc Update call real diff from last update and use it."

This reverts commit 10784a8c7cc81c468b5411e973d36ecf31de9603.

Main reason: impossibility for me as commiter test problem and fix all corner cases problems.
Fixed mangos.sql `creature_template` default content after adding vendor_id
Fixed creature_template loading after vendor changes.

Thanks to insider42 for problem research.
Update outdated comment
UInt32 timer class copies instead using time_t timers for mstime cases.

Also cleanup weather update code and some random code cleanups.
New version of patch for send real diff from last update.

In new version last update time stopred for specific Cell that store all world objects
placed in it. All objects of Cell updated (or not updated) in same time.

Original version provided by ciphercom.
Restore some item checkes for temporary loot case.
Fixed *nix build problem after recent commits.
Add dummy effect of spell 53475, 53487 and 54015
Fixed some GCC warnings and code errors.

Thanks to freghar for provide cleaned list of warning messages.
Fixed wrongly use %d for guids in DB queries.

This can affect correct result in cases big guid values
(out of range for possitive part of int32 range) at x86 platforms.

Some other cases addded in addition to original patch fixes
where %d wrongly/unsafe used for unsigned values.
Update SpellEntry based on the work of TOM_RUS
Use portable ACE way msecs time value calculation

Avoid use unsafe code for non-Windows platforms
Thanks to Marik for research the problem and Ambal for helping me with this patch
Cleanup some death state enums usage.
 Avoid selection despawned creatures as spell autoselected targets.

Original patch provided by virusav.
Less strict check for update at war state for factions

Clarify in comments of recent added spell dummy effect
Compute cell area bounds for active objects in the same way like it done for players
Add debug note for not implemented summon property group
Remove unused structure GridUpdater.
Correct check percent spell costs in AI::CanCast

Also use uint32 for spell cost fields/results
Fixed loading mails broken after [10692]
Check SPELL_ATTR_IMPOSSIBLE_DODGE_PARRY_BLOCK for special attacks.
Add the option to close gossip window in gossip system

Field action_menu_id in gossip_menu_option table can be set to -1 where close gossip is expected, instead of sending a new menu.
Simplify CellArea calculation
Implement talent 30326 for spell 5138.
Move Glancing Blow early in melee outcome results.
Update ACHIEVEMENT_CRITERIA_TYPE_LOOT_* with group roll.
Prevent wrongly use transport guid as unittarget in spell targets
Fixed typos in my previous commit
Check localization data for existance main table entres.

Data will skipped for nonexistent entries, and reported with
disabled LogFilter_DbStrictedCheck filter.
Startup checks for `gossip_menu` and `gosip_menu_option` table data.

Existance `gosup_menu` id refercned from gameobject field and unused meni ids
activated only with disabled LogFilter_DbStrictedCheck.
player/group instance binding fixes.

* Use anum instead raw uint8 type in args
* Fixed crash when gm at continent invite to group gm in instance
  and then teleport to instance using .goname.
  When group leader teleport to instance it must get group bind instead solo bind.
* In other semilar cases detection report error as before but replace solo by group bind instead
  assert crash at enter to map.
Avoid unexpected error at action_menu_id < 0 after recent loading checks adding.
Fixed move in inventory items with ItemLimitCategory.
Typo show commit before revet all realddiff one more time.

Typos show that in fact "working" has been result typos diabled state for relamd.
Cell-based way have not avoidable problems with case when object added to not active
long time Cell. If someone find more working sollution it can be readded ofc.
Revert "[10716][10688] New version of patch for send real diff from last update."

This reverts commit 8398a55fa274471daae115e00c627b299a3fdbbd.
This reverts commit 06e2d6859ba3d7fd47be72c23a64e68ae039701f.
Partly revert "[10687] UInt32 timer class copies instead using time_t timers for mstime cases."

This reverts commit 967877fefd91e00895f9790dc006b532b94d41b7
in part MapManager/World timers.

Main reason: unclear source cases wrong mstime diff generation at WorldRunnable level at *nix systems.
So need have existed in past hacks that skip its.
Implement proper calculation quest honor reward.

Now server side rewar same as reported at client side.

Also thanks to KiriX for patch prepering to commit and testing.
Implement check `gossip_menu_option`.`npc_option_npcflag`.

For cases main menu options used by creature but impossible show in result absent
related npcflags for any creatures used that menu.
Use required_quest instead of _heroic for non_instance areatrigger teleport
Fill target map for TARGET_RANDOM_NEARBY_DEST(b) cases.

Corrects ~35 spells, where most has summon effects.
Backports from 400 branch.
Add dummy aura effect of 51405 and dummy effect of 51420
Fixed wrong error message for quest related areatriggers.
Fix TARGET_AREAEFFECT_CUSTOM, need to obey type when DB target entry exist
Add a few comments for SUMMON_PROP_TYPE_CRITTER
Correct one SQL in tools, table was renamed.
Map system re-engineered. Special thanks to Blueboy for tests.
Fix Deathly Stare spell
Add dummy aura effect of spell 47178 and dummy effect of 47176

Note: see comments in code for spell 47176
Check if Pet's target is a valid target (isTargetableForAttack)
Remove redundant isAlive-check, thanks LordJZ for notice
Not send session fields as args into HandleMoverRelocation/VerifyMovementInfo.

Also mark args as constant where expected in this functions.
Implement defencive guardians pet type for creatures like 28120
Rename and clarify SummonPropertiesEntry.Type->Title field meaning.

This also show why we have some problems with sorting sumons by types base at this field.

Thanks to TOM_RUS for research.
Fixed spell 51420 work by use recently added protector pets as expected.

Thanks to NoFantasy for problem research (and prev. new pet type adding case research aslo) ;)
Fixed  unexpected typo in prev. commit.

YTDB 351

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

[Dj Aligator]

Mangos 10738 - 10777

In fact set pet name to empety state redundent.

Client work propertly also if used default name from creature template (set in create code).
Cleanup pet remove code and some fixes.

* Merge Player::RemovePet and Pet::Remove function code to Pet::Unsummon
  This let be sure that in all cases all required steps doen. For example this fix
  creature's guardians propertly remove from guardians list.
* Add new pet save mode PET_SAVE_REAGENTS as replacement PET_SAVE_NOT_IN_SLOT+true-arg stable pair in old function args
  This will avoid use reagent save arg with wrong different save modes.
* Fixed recently added code with absent check for re-summon protector pet call.
 Cleanups in spell effects names.
Add AI function OwnerKilledUnit for protector pets (when owner kills unit)
Add `spell_bonus_data`.`spell_bonus_data` field and support for RAP bonuses use for spells damage from table.

Thanks to insider42 for prepering patch for mangos repo.
Also added loading check for redundant ap/ap_dot data in table.
Add AP/RAP bonus data to `spell_bonus_data`.

Also remove related hardcoded cases.
Prevent equip/unequip items in process logout
Use ObjectGuid in spellauraholders instead uint64

(based on rsa's repo commit 06342c1)
Use objetguid for other .list auras command part
Some other aura caster object guid use cases.
Convert MonsterSay/Say and similar functions to ObjectGuid use.
Convert Gameobject owner to objectguid use.
Convert Gameobject owner to objectguid use.
Use objectguids in item code.

Also
* Fixed unexpected container updated state after inventory load
* Make .debug getitemstate command more friendly by args.
Convert to objetguids map scripts and finally drop IS_PLAYER_GUID.
Lost mangos.sql changes in one from recent commits.
Use UNIT_FIELD_CRITTER for store minipet guid.

Also move related code to Unit and allow summon minipets by creatures.
Also fix creature's totems unsummon at owner remove from world.
Use class specific combat rating factors from gtOCTClassCombatRatingScalarStore.dbc

* Some classes had effect of melee haste buffed
* Change to armor penetration rating was hidden there too
Implement armor penetration limitations

The tool-tip says "up to X%" for a reason...
Implement zone limited pets unsummon at leave allowed zones.

* Also remove redundant zone/area args for zone/area update function like UpdateZoneDependentAuras.
* Merge Player::*AllControlledUnits functions to Unit version with replace option args by mask arg.
* Unit::*AllControlledUnits guardian iteration make working with internal unsummon call suport for guardians case.
Fixed typo in CONTROLED_*
Correctly reset mob orientation upon reaching spawn point. Should also improve visualisation of mob orientation on clients
Remove unneeded non-player Say et al.

Now expected direct use non-player WorldObject::MonsterSay/etc version
Basic support for target-name in MonsterSay/etc

Change MonsterSay's target to pointer.
Implement SPELL_AURA_IGNORE_UNIT_STATE (252).

Original patch provided by Wojta

Note: From patch excluded talent 44543 and ranks stacking code.
      It look like need another implemention different from suggested,
      or at least more research.
In patch added enum IgnoreUnitState and related checks now more cheating protected.
Move comments at 61 line, no functional changes.
Implement talent 44543 and ranks charges work.
Also fix IGNORE_UNIT_TARGET_NON_FROZEN in part allow casting spell that required frozen target.
Implement SPELL_AURA_MOD_PERIODIC_HEAL (259)
Allow AURA_FORCE_MOVE_FORWARD to work on creatures and vehicles. For example, it fixes spell [68284]
Show <overkill> value in combat logs.
Remove useless code in PetAI::_stopAttack()

Those are all processed elsewhere and _stopAttack() can not be called when pet is not alive.
Fixed trainer flags uses for pet trainers.

* Not exclude trainer flags for client for pet trainers.
* Not report error for pet trianers empty spell lists.

TODO:
* Add check for redundent trainer spell data for pet trainers at server startup
* Need hide train gossip option for pet trainers, but currently without it gossip
  window not open for pet trainer by some unknown reason.
Fixed auction items filering by quality.
Explictly include Threading.h into WorldRunnable.h

Instead dependent from includes order in mangosd\WorldRunnable.cpp and mangosd\Master.cpp
Fixed build for different PostgreSQL version at diff platforms.

* Fixed Problem in different place define OID enums.
  Enums hardcoded in mangos code now as done for many other projects using PostgreSQL

* Other PostgreSQL detection fixes.
Fixed crash at empty build list in realm string.
Implement more generic reputation spillover

* Also fixes factions 1037 and 1052
* More correct client output when reputation assigned to parent faction
* reputation_spillover_template can still be used to override spillover from dbc
Use separate vars to indicate whether reputation needs to be sent or needs to be saved.
Not mark `reputation_spillover_template` empty state as error

Now expected by default data used from dbc fields.
Explictly include Threading.h into CliRunnable.h

[Dj Aligator]

Mangos 10778 - 10792

Make AI call OwnerKilledUnit() more proper.

Affect any pet that are not being controlled (minipet/guardian/protector) and will also apply to pets owned by another creature.
Allow ITEM_ENCHANTMENT_TYPE_COMBAT_SPELL to proc on special attacks too.
Fixed warning in ReputationMgr::SetReputation.
Use BattleGround(AV)TeamIndex enum types in AV battleground code.

* Added BattleGroundAVTeamIndex enum type for 3-case special team list fo AV.
* Rename BattleGroundTeamId -> BattleGroundTeamIndex for clarify
* Some safe checks added and dropped bogus call RewardReputationToTeam in BattleGroundAV::HandleQuestComplete
  for BattleGroundTeamIndex as faction. Unclear for what it expected used so dropped.
Use Team enum types in all appropriate cases and catches bug in result fix.

* Fixed wrong arenaid use at leave arena queue.
* Fixed memory lost and etc at not virtual EndBattleground call
* Fixed crash at arena join with fake data from client.
* Code cleanups.
Fixed comment in characters.sql
Restrict items with generated loot really to single character only.
Fixed warnings at comparison BattleGroundTeamIndex and BattleGroundAVTeamIndex

Maybe adding special operator== or != can be more nice way but for small used cases this
not look so need really.
A bit spell proc housekeeping

* PPM values for enchanting spells 13898, 20029, 20032, 44524, 44576
* Correct cooldown for enchanting spell 59625
* Add procEx and cooldown for item 50198
Add explicitly header dependences for Database/QueryResult.h
Replace bytes in single field `bytes2` by independent fields for `creature_addon`.
Allow use old non-builting ACE versions (pre-ACE-5.5.4) at Unix-side.

This fix build error in relamd code in result
nonexistence ACE_OFF_T in old ACE versions.
Moved vmap code from shared to game project

After this, realmd code does not contain anymore vmap code
Small cleanup in VC100 filter about vmap files
Fixed more warnings with BattleGroundTeamIndex/BattleGroundAVTeamIndex mixed use.
Fixed warnings and need in redundent char* casts in database access code.

YTDB 354

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

[Dj Aligator]

Mangos 10793 - 10834

Fixed build problems for vc90 after recent vmaps dir move.

VC80/VC100 not affected by this.
Player.h expected DBCStores.h including.

So make it direct.
I must less press random buttons before push in monkey mode.
Use regular map case in all trigger checks.
Check for valid sheath state in creature addon at startup.
Some more warning fixes related to BattleGroundTeamIndex/BattleGroundAVTeamIndex.
Add GetGuidStr() short call for GetObjectGuid().GetString() and use it.
Use ObjectGuid in some battleground structures.

Also fix some catches bugs in code in result.
More ObjectGuid uses in battleground code.
Use ObjectGuid in Mail code.

Some catched bugs fixed in mixed uint32/guid values using cases.
Use ObjectGuid in player loading code.
Restore build at *nix.
Remove code duplication in visibility distance check.

at_same_transport already include player checks at assign and
it move to common if make 3 distance cases exactly same.
Fixed typo in m_PrematureCountDownTimer initila value set.

Just fo note: in fact initial value assign not used because
m_PrematureCountDownTimer always init and used only in case
m_PrematureCountDonw == true
Typo fix and memory layout optimization.
Small cleanup and optimization in transport current point update.
Remove diplicate World::GetQueueSize and rename functions to consistence naming
More ObjectGuids specially in petition code.

Some uint64<->uin32 bugs fixed at read guids from DB in petition code.

Still exist 29 GUID_LOPART uses in code.
Use ObjectGuids in chat commands.

Still exist 15 GUID_LOPART uses in code.
Use ObjectGuid for corpse owner guid.

Still exist 9 GUID_LOPART uses in code.
Use ObjectGuid for aura holder cast item guid.

Still exist 7 GUID_LOPART uses in code.
Convert to ObjectGuid aura cast item guid access function.
Use in social lists API ObjectGuid form.

0 GUID_LOPART uses in code. :)
Remove GUID_LOPART/GUID_HIPART

* Now no correct way access to internal structure of guid in uint64 form.
  For this always must be used ObjectGuid form.
* Guids in uint64 form still used in some cases (many) but it uses as-is
  Main source of uint64 guids is non-converted opcode handlers.
* Also ofc, possible cases wrong created uint64 guids by assign uint32 to uint64.
  For catch all cases need finish convertion to ObjectGuid all non-converted yet code.
Add explicit include for ObjectGuid.h in AchievementMgr.h.
Add dummy effect of spell 47170
Fixed creature's speed recalculation, forgotten DB speed mods are used now

Also properly update speeds at creature entry change(in case there are aura speed bonuses), thanks to Vladimir for pointing
Speedup and cleanup quest counters update code.

* Always use QuestStatus data for current counters set
  This let be sure that that we not have non sync values in client view and server state.
* Remove some access functions for quest update fields that expected to be only updated/set.
Restore anti-cheating check work at buy item
Fixed "unsigned is never < 0" warning.
Clear way access/set UNIT_FIELD_BYTES_*/PLAYER_BYTES* byte fields.

Fix some bugs when all uint32 field modified instead byte parts.
In some cases added byte way set/get but code line itself need additinal reserch/fix
for use correctness in line pos.
Fixed copy-paste typo in prev. commit.
Move specific SQLStorage defines to game dir

This let have in shared dir only generic declarations of class SQLStorage
Not game specific details.
Fix vmap_assembler compilation after vmap folder move

Also make vmap_assembler compile sucessfuly with VC80.
Add VC80 projects and solutions files where missing like vmap_extractor
Fix warnings for TBB projects compilation using Visual Studio 10

Also fix a small typo in prev commit.
Update SpellFamily check functions.

* Move SPELLFAMILY_* to DBCEnums.h and name it as SpellFamily
* Use enum in Unit::GetAura, move family mask code to new SpellEntry helper function.
* Avoid all auras scan in search of triggred spell for 20243.
 Prevent spell_proc_event entries to proc on any spell cast when SpellFamilyName is zero.

Now only spells that actually deal damage or heal will trigger
when neither SpellFamilyName nor ProcEx & PROC_EX_EX_TRIGGER_ALWAYS are set.
Before, setting a SchoolMask would override this check, causing many bogus procs.
Fixed crash at send mail from console

Inspired by patch from rsa.
Spawn pool at aura 63471 dispel in water.
Avoid use unsafe remove auras in aura apply code.

Specailly example Aura::HandleModCharm code where old code
call RemoveSpellsCausingAura or (a) dead and not called,
or (b) will remove aura self at call.

* Added safe for aura holder context version of RemoveSpellsCausingAura
* Cleanup old simple RemoveSpellsCausingAura code also.
 Cleanup shapeshift form code.

YTDB 355

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus
+ new templates from WDB
+ processed 46 caches (4.0.1)

[Dj Aligator]

Mangos 10835 - 10866

Add/correct a couple more spell_proc_event entries for trinket related spells

*Affected items: 45131, 45219, 50359, 50366, 50360, 50365,
          50353, 50348, 50362, 50363, 50342, 50343, 50358
Again ignore EffectSpellClassMask when SchoolMask given in spell_proc_event.

There's just too much garbage in dbc...
Should restore some damage school based talents broken with [10830],
hopefully without excessively triggering other spells again that shouldn't.
Implement script effect of spell 26218.

Also do small spells ordering around.
Not report error for missing reqs for some quest timed achievements.

This achievements in fact not required any additional checks.
At far teleport fail retunr player to source point instead homebind.
Implement server side part of SPELL_AURA_ADD_MECHANIC_ABILITIES (293) work.

Also fix stealth/invisibility visual affect flags changed at some 3.x switch.
Implement charges counting for magnet target auras.

Also implement skip spell effects redirecting if magnet target can't be
targeted by this spell effect base at spell effect target data requirements.
Allow fill DB table spell_script_target for TARGET_AREAEFFECT_INSTANT(7)

Without any database entries for spell, spell will work like before.
Valid targets filled to database can only be creature type (corpse or alive).
Fix AuraType 138 to not affect ranged haste, and rename to SPELL_AURA_MOD_MELEE_HASTE.

Also type 192 had misleading name, rename to SPELL_AURA_MOD_MELEE_RANGED_HASTE.
Remove small redundency in magnet spell charges checks.
Add dummy effect of spell 49634 and dummy aura of spell 56511

note* spell will need data in spell_script_target to work properly.
note* the effect of 56511 does not work, see comment in code for details (code added in any case, to not forget the problem)
Add dummy aura of spell 43874 (indirectly related to spell/effect 50133)
Unsummon guardians at second item use for items without cooldown.
New system for parallelizing client packet processing. Handle WorldSession updates in Map::Update() where we are safe to proceed. Thank you for all your feedback!
Fix build with gcc.
Move common check part for recently added filter classes to helper function.

This is more safe have expected tio be same check in one function.

Also apply some code style fixes.
Small function order in sources update.
Fixed typo in [10847]
 Report about GM invisible state at login if enable.
Fixed crash at .character erase command use.
Add TARGET_AREAEFFECT_GO_AROUND_DEST(52) (renamed from TARGET_AREAEFFECT_CUSTOM_2)

Target selects all gameobject around destination, limited by adding spell with a corresponding gameobject entry in database table spell_script_target.
Correct two whitespace errors
Complete set Byte/ShortFlag value functions and use it for PLAYER_FIELD_BYTES proper access.

Not expected any changes in work.
Add triggered spell part for dummy effect of spell 43572
Search for linked GO using the original used GO as base of the search

Add some comments for future improvements of this function.
Handle GAMEOBJECT_TYPE_TRAP in Use, as done for other GO types.

This just move parts of existing code from TriggeringLinkedGameObject and does not affect summoned trap type GO's (or related to summoned).
Large flightless birds (emu) think they are invisible, even when they are not.

Only if _not_ isGMVisible, invisible notification can be sent :)
Fixed not show and lost mail items in returned expire mails.

Also
* Use typename for mail item info vector.
* Include tools/characters_mail_items_cleanup.sql for cleanup lost mail items.
  It also included as sql update part. Sql update also fix receiver for stored
  in DB expired mail items.
Fixed triggering/use effects for item 54589, 54573, 37734, 32500, 28590.
Implement itemsets 881, 882 at heal/damage target effect.
Make DB code thread-safe. Original patch by Machiavelli and Kero99.
Unsummon guardians at re-cast only for non-triggered case.

YTDB 356

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus
+ new templates from WDB

[DarkVanir]

Fantastic work. I actually dislike MaNGOS, but your work is awesome, thanks for the compiles, etc. Rep added.

[Dj Aligator]

Mangos 10867 - 10894

Implement templates for trainer spell data in `npc_trainer_template`.

New table let avoid duplication similar trainer spells for different trainers.
Fixed spell 53479.

(based on insider42's repo commit b0343e1)
Fixed typo in mangos.sql added in recent commit.
Properly remove deleted character from friends of online players.
Avoid lost arena match at leave queue (before invite to match)
Grr, restore code style.
Fixed crash for case when trainer have only data in npc_trainer_template.

Good know base at report that this feature so quick start used actively ;)
 Update ACE to v5.8.3

(based on zergtmn's repo commit 3a8c259)
(based on zergtmn's repo commit 946c1a8)
 Update ACE version in NEWS/ChangeLog
Really fix crash in SendTrainerList :/
Traditional 'Waypoint' post-fix in mangos.sql
Update achievement at maximize skill level by command or config option.
Remove support unused option SocketSelectTime in mangosd.conf
 Avoid constant re-save BG entry data at each save for player at bg/arena.
Lost important line in prev. commit.
No error output for empty npc_trainer* tables
Implement item 49982, 50641 equip effect
Pet with same level as owner must have xp = 0

Thanks to Click for original patch and problem description.
Hunter pet can't have < owner-5 level
Also make tame time check call safe.
Fixed crash at GO use in case GO owner/first user offline state.
Add dummy aura effect of spell 42515
Use 0.0f in one function and prevent some compiler warnings.
Add dummy aura effect of spell 42517
Merge apply/remove cases for recently added dummy auras.
Access for core/scripts to creature race info in model dbc
Fix load creature *addon data from database
Speedup check code at `gossip_menu_option` loading.
Fixed problems with new ACE lib on Darwin systems

(based on cipherCOM's repo commit ce2638f)

YTDB 357

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

[Dj Aligator]

Mangos 10895 - 10944

Description for found gender field in CreatureDisplayInfoExtraEntry.
Detect self-referenced model info in `creature_model_info`.
Small typos in error message.
Fix for pool system problems after map rewrite.
Remove default value for removed config option SocketSelectTime.
Remove SPELL_AURA_MOD_POSSESS_PET auras in Unit::Uncharm()
Allow vehicles to have gossip menu

In addition rename IsGameobject -> IsGameObject.
Simplify code for triggering linked trap
Drop unused code in Unit::MeleeDamageBonusDone()
Cleanup character_battleground_data when deleting characters from database
Fixed vendor templates work.

* Avoid unexpected errors at template loading for shared templates for many npc.
* Really use vendor template in work.
Fix Glyph 45806 to trigger from 49184 and ranks instead of unrelated spells.

Also fix attack power coefficient for 49184 and ranks.
Allow mounts on map 603
Imporve detection need use reagents by triggered spell.

Now regagent used if main spell (if infor about provided to triggered cast)
not have regent data.

This fix reagetn use for spell 43723. But maybe need another fixes for proper animation.
Code style cleanups in scripting related functions
Use provided target instead of caster for SPELL_EFFECT_QUEST_COMPLETE
Fix GCC warning
Move scripting related functions from ObjectMgr to ScriptMgr
Fix db_script_string loading
Remove ObjectLevelLockable inheritance in Map class + cleanup locks in Map code. Restore build after recent commits.
Use HasFlag function for check flags instead raw access.
Restore GameEvent system work after last Map terain rewrite.
Implement dummy aura for spell 75973
Allow SPELL_AURA_MOD_PACIFY auras to affect creatures
Add support for ITEM_MOD_SPELL_PENETRATION
Actually use spell penetration gained by items in calculations
Prevent server freeze at strnage radisu used by item 34124 spell cast
Allow from mute player chat messages CHAT_MSG_AFK and CHAT_MSG_DND.
Revert "[8722] Fixed possible creash."

This reverts commit 8dd3e4b31a31bb6dfc4d9fb51b1b7e67bad4ca16.

This is impossible crash case. Creature object can't be created without creatureInfo != NULL
So crash source can be only result wrong calling ClearInCombat() for deleted or NULL creature.
Cleanup some magic numbers
Send time diff between Update() calls for object - should help with mob respawn on inactive grids etc. Based on patches by VladimirMangos and cipherCOM. All issues from previous patches should be finally fixed now.
Few more vehicle flags

All vehicles with these flags have corresponding movement flags.
Remove player-only limit for HandleAuraTransform where miscvalue is 0

Add error output for unhandled spells plus minor code cleanup
Prevent null-pointer dereference in AI calls
Uncomment all races in enum Races

Race can also be creature race and needed for future implementation of spells.
RACE_UNDEAD_PLAYER renamed to more proper RACE_UNDEAD
Move game event creature morphing code to Creature::UpdateEntry

* This is allow have more clean logic in feature work.
* Prevent modify static creature data.
Add models for spell 65528 and 65529

Add spells in commented form that will need custom handling in similar way for the same aura effect.
Fix mangos build with old ACE libraries after [10924].
Imporvments gameevent creature morphing.

* Table `game_event_model_equip` renamed to `game_event_creature_data`
* Table allow now store same creatures for different events, BUT
  expected that related events no active in same time.
* Added possibility switch entry at gameevent time.
  This let have diff factions/loot and etc for creatures.
* Added possibility cast spells at gameevent start/end.
  Exist some spells that expected casted to creature at gameevent start
  for model replace, and for animation in other cases.

Note: `game_event_creature_data`.`modelid` field posisble will removed soon in fowor related spell use.
      Ofc, when spells will implemented in core.
Fixed crash at modified data from client at trainer spell learn.
Implement dummy effect of spell 7671.

Thanks to X-Savior for reserch.
Move script calls to ScriptMgr

- Script library presence is now optional.
- Some script hooks have new names. Scripting libraries need to be adjusted accordingly.
Correctly show online status of 1.12.1 realms
Add models for spell aura 50517 and 51926
Some adjustments to script calls OnQuestChooseReward replaced with OnQuestRewarded. Also fix compile in VC 8.0 and VC 9.0.
Check non-unit target case for avoid crash in EffectQuestComplete.
Apply mangos code style to in class {} indent.
More strict check player race/class allowed values.
Implement .reload npc_text.
Fixed mangos.sql after rename table to `game_event_creature_data`
Lock action buttons before unlearning talents on spec switch.

This prevents client from automatically requesting removal of buttons that use unlearned spells.

YTDB 359

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus

+ all fixes from forum
+ fix spawn
+ quest scripts
+ mob's gossip menus
+ new templates from WDB

[St3w13]

Respect. ***** 5/5 stars .