Jump to content

[11140]Achievement reward: faction related titles


Guest Shadez

Recommended Posts

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

Properly reward faction related titles to player when achievement has been completed (currently just one - 100k honor kills - %s of the Alliance / %s of the Horde).

* For which repository revision was the patch created?

11139

* Is there a thread in the bug report section or at lighthouse? If yes, please add a link to the thread.

No

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

Me

diff --git a/src/game/AchievementMgr.cpp b/src/game/AchievementMgr.cpp
index 2a0269d..7b4cdfa 100644
--- a/src/game/AchievementMgr.cpp
+++ b/src/game/AchievementMgr.cpp
@@ -2086,7 +2086,7 @@ void AchievementMgr::CompletedAchievement(AchievementEntry const* achievement)
        return;

    // titles
-    if(uint32 titleId = reward->titleId[GetPlayer()->GetTeam() == HORDE ? 0 : 1])
+    if(uint32 titleId = reward->titleId[GetPlayer()->GetTeam() == HORDE ? 1 : 0])
    {
        if(CharTitlesEntry const* titleEntry = sCharTitlesStore.LookupEntry(titleId))
            GetPlayer()->SetTitle(titleEntry);

Link to comment
Share on other sites

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use