Update council every time group is updated.

If a council member joins late, there was no quarantee that they would get picked up by the system, leaving them without voting frame. This ensures the council is updated when the group is, whilst also being a small performance gain.

changelog.md

@@ -1,3 +1,10 @@
+## Changes
+
+
+### Bugfixes
+
+- *Council wouldn't always be registered properly.*
+
 # 3.14.5
 
 ## Changes

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