Core/AchievementMgr: fix integer underflow neglecting already completed realm firsts

Closes #19334

(cherry picked from commit 9cfa7c752e)
This commit is contained in:
ariel-
2017-03-19 19:33:30 -03:00
committed by funjoker
parent 848f0d1323
commit 786bb561ca

View File

@@ -1016,6 +1016,9 @@ bool AchievementGlobalMgr::IsRealmCompleted(AchievementEntry const* achievement)
if (itr->second == std::chrono::system_clock::time_point::min())
return false;
if (itr->second == std::chrono::system_clock::time_point::max())
return true;
// Allow completing the realm first kill for entire minute after first person did it
// it may allow more than one group to achieve it (highly unlikely)
// but apparently this is how blizz handles it as well