UpdateAchievementCriteria logs

[charlie2025]

What bug does the patch fix? What features does the patch add?

In function UpdateAchievementCriteria() there is one output - sLog.outString , This spam console a lot, from one player there are few more lines in console at basic loglevel set.

So changed this log as detail.

For which SubVersion revision was the patch created?

master - 0d2b8e0190654f38e8816343fbba5b5839a4d4b0

Who has been writing this patch? Please include either forum user names or email addresses.

Charlie2025

git pull git://github.com/charlie2025/mangos.git achiev_log

( http://github.com/charlie2025/mangos/commit/61d72051da5de9618d5e325f044b3ed22665a56c )

[Ebrithil]

Thx, so much console spamming at lv log 1 was very annoying

[xeross]

Vote for addin or something similiar

[ApoC]

Applied in [6974].

Thank you.

[oc_redfox]

Applied in [6974].

Thank you.

Maybe changed this also?

sLog.outString("AchievementMgr::SetCriteriaProgress(%u, %u)", entry->ID, newValue);

to

sLog.outDetail("AchievementMgr::SetCriteriaProgress(%u, %u)", entry->ID, newValue);

[charlie2025]

Maybe changed this also?

sLog.outString("AchievementMgr::SetCriteriaProgress(%u, %u)", entry->ID, newValue);

to

sLog.outDetail("AchievementMgr::SetCriteriaProgress(%u, %u)", entry->ID, newValue);

Hmm, I had biggest spam from UpdateAchievementCriteria, but yes, other should be set too.

http://github.com/charlie2025/mangos/commit/c80d2094f0007092e371a923d288e814de216f33

Set also other output in achievements to detail

[marry521]

LogLevel = 0

has the prompt unceasingly!

[marry521]

http://github.com/charlie2025/mangos/commit/c80d2094f0007092e371a923d288e814de216f33

[charlie2025]

http://github.com/charlie2025/mangos/commit/c80d2094f0007092e371a923d288e814de216f33

Fixed in 7014

[oc_redfox]

Hey charlie... Open a new thread maybe?

Not to sure if the devs are checking for patches in "Accepted"...

[charlie2025]

Hey charlie... Open a new thread maybe?

Not to sure if the devs are checking for patches in "Accepted"...

Fix isn't needed now, Arrai done same changes in 7014