[patch][6735] Group Roll Choice Visual Fix

[krz]

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

It fixes visual bug before real roll.

Now you will see:

"You/playersnick have/has selected Need/Greed for [item]"

Instead of

"Need/Greed roll - 1/2 for [item] by playersnick.

For which SubVersion revision was the patch created?

6718

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

I don't know

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

krz

Index: src/game/Group.cpp
===================================================================
--- src/game/Group.cpp        (revision 6718)
+++ src/game/Group.cpp        (working copy)
@@ -623,14 +623,14 @@
                break;
                case 1:                                                                                         //player choose Need
                {
-                        SendLootRoll(0, playerGUID, 1, 1, *roll);
+                        SendLootRoll(0, playerGUID, 0, 0, *roll);
                        ++roll->totalNeed;
                        itr->second = NEED;
                }
                break;
                case 2:                                                                                         //player choose Greed
                {
-                        SendLootRoll(0, playerGUID, 2, 2, *roll);
+                        SendLootRoll(0, playerGUID, 128, 2, *roll);
                        ++roll->totalGreed;
                        itr->second = GREED;
                }

[Sephiroth1983]

Was annoying to have those 2 rows of same roll with same nick, strange...

thanks

[Xlayer]

Yeah, good patch, this improves loot readability a lot, thanks.

[Klaimmore]

Didn't tested it, but according to the comment on the current function definition and the fact that the function only send the information to the client, there isn't much to test. If that worked as expected for you, then it definitely works.

[VladimirMangos]

In [6735]. Thank you