diff --git a/changelog.md b/changelog.md index b0212f81..e941f8dc 100644 --- a/changelog.md +++ b/changelog.md @@ -1,3 +1,10 @@ +## Changes + + +### Bugfixes + +- *Council wouldn't always be registered properly.* + # 3.14.5 ## Changes diff --git a/ml_core.lua b/ml_core.lua index 5f0dcd64..a1de3d23 100644 --- a/ml_core.lua +++ b/ml_core.lua @@ -416,6 +416,7 @@ function RCLootCouncilML:OnGroupRosterUpdate() if newGroupSize > self.groupSize then self.Log:d("Group size changed to "..newGroupSize) MLDB:Send("group") + self:UpdateGroupCouncil() self:SendCouncil() end self.groupSize = newGroupSize @@ -1309,15 +1310,12 @@ end -- Used by the ML to only send out a council consisting of actual group members. function RCLootCouncilML:UpdateGroupCouncil() Council:Set{} -- Set empty + local candidates = addon:UpdateCandidatesInGroup() for _, guid in ipairs(addon.db.profile.council) do -- REVIEW: Is all this Player:Get() really efficient? - local player1 = Player:Get(guid) - for cand in addon:GroupIterator() do - local player2 = Player:Get(cand) - if player1 == player2 then - Council:Add(player1) - break - end + local player = Player:Get(guid) + if candidates[player.name] then + Council:Add(player) end end -- Ensure ML (us) are included