* Use min/mac level from dbc because anyway list real selectable
for player level bg/arena backets set min/max range.
* Drop related fields and update per team min/max values.
In part related broadcasts by explicit provided Map pointer (in different object GetMap() requested)
This is must fix recent crashes in Map::MessageBroadcast
We not have currently way set proper set skill step at loading,
but this not required, because important only for new learned rank have different step.
Also drop outdated (as tested) comment.
* Move step arg in SetSkill to end for make possible for it default value.
* Merge GetSkillStep function to SetSkill, base at fact that old GetSkillStep
code can be != 0 only for already known spell, so at update
* Avoid use explicit MailReciever/MailSender where not need.
Types specially added for autoconvertion from common cases.
* Prevent allow copy text auction mails. It anyway store encoded data.
Same for some another non-player mails.
This is must decrease amount of data send to network
at player enter to visibility radius for another player.
(based on SilverIce's repo commit 8c60a80)
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
(based on SilverIce's repo commit e226019)
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Note: this changes not affect in any good/bad way recent problem with player guids.
This is resolve "none from players see each other" problem.
Please, reapply non-0 player guid only with or after this problem resolve.
Some ObjectGuid use cleanups added while reseach porblem source.
Required DB data for real work. This is final part required at core side for support.
Also note that commit include
* generic implementation money cost gossip menu items so let
implement in DB many money cost gossip operations with _fixed_ cost.
* useful CONDITION_SPELL that let implemtn menu item visible if specific
spell known/don't known also wide used case for diff gossips.
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
* At login reset '.reset all talents' will reset all spec talents.
* New command '.reset specs' will reset its online/offline.
* Command '.reset talents' now not support offline player case.
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Also add check to prevent have NONE gender with some other for same entry
in `achievement_reward` and `locales_achievement_reward`.
