This fix problem when by same order case left/right auction greater each other
in same time. This possible in old code because selection way compare for
case dependent only from one from compared auction so it can be true/false
for left/right.
* Remove excluding normal repeatable quests not first cases for money/xp
rewards. This is not confirmed functionality and possible related wrong
repeating flag set for "first" repeating quest versions. Whern exist
normal quest and second repeating quest with same details.
* At max level positive RewOrReqMoney already included in RewMoneyMaxLevel
and then not need rewarded\
* When RewOrReqMoney > RewMoneyMaxLevel then RewMoneyMaxLevel values
ignored and always used RewOrReqMoney as money reward.
Thanks to Lynx3d for original problem research.
"-a configfilename" or "--ahbot configfilename" can be used for this
This possibility can be useful if normal mangosd.conf path also set by -c
option for example.
Update it in your habot.conf version if you have haby load at mangosd
startup time from ahbot activity
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Source crash in missing locale strings array size check before access
to it in locale structure.
Also move repeating code for access to wide used localization string arrays to ObjectMgr functions.
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
Also
* Prevent more one charge use for same spell cast
* Cleanup enum SpellState from unused cases
* Propertly remove spellmod charges at spell finish in case pet/totem caster
Signed-off-by: VladimirMangos <vladimir@getmangos.com>
By unknown reason added in last weeks or longer period gameobject
visually moved only after relogin. GO removed from client view and readded
at same place _but_ have and send to client new coordinates as debuged.
This problem researched and not comnnected directly with fixed bug.
TODO: proper solution would be update creatures while in corpse state.
This also would solve problems with death persistent auras that not get updated for died creatures
This cleanup small code and let catch some missing check cases
when 0 guid attempt searched by real DB query.
This also make function call more safe if it used with non-player guid by
some reason.
