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.
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Grr... git_id tool corrupt old sql update (it drop sometime all except guards)
and i copy old not fixed version for restore :/
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
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>
It included in swith time sql update for use in cases not converted yes characters DB.
It can be not 100% safe used for already used in 3.3.3 mode chaarcters data.
BUT if you want you can use it at own choice ofc.
Fixed durability problem after switch. Converted non-bag items only.
Bags not have normally item text and durability.
* 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`.
* character_account_data and then will partly included macros/settings local for character.
Mostyl used account wide macros not included.
* character_declinedname
* character_equipmentsets
* character_glyphs
* character_pet_declinedname
* In beggining bad news: at 3.3.3 switch has been forgoted item_instance.data field chnage.
In result items with texts (mail copy in to inventory, some other) lost text content.
* That why in current commit item_text droped without convertion. In result all itesm with texts
will lost text. Sorry for this as i think small porblem :(
* Now text stored in item itself.
Also add `character_talent` including to pdump
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Also drop save `item_text` as not used in items/mails now after switch to 3.3.3
* Quests.Daily.ResetHour (0..23) for daily quests
* Quests.Weekly.ResetWeekDay (0..6) and Quests.Weekly.ResetHour (0..23) for weekly quests.
* Drop time from character_queststatus_daily and use same way as weekly quest reset time store.
