evil-morfar / RCLootCouncil2

RCLootCouncil - addon for World of Warcraft
https://rclootcouncil.com
GNU Lesser General Public License v3.0
19 stars 29 forks source link

Attempt to fix mldb #107

Closed SafeteeWoW closed 6 years ago

SafeteeWoW commented 6 years ago
evil-morfar commented 6 years ago

The problem (why I removed the check) was that the ML could send the mldb before users registered him as ML. I suppose now that the groupleader is always registered as ML, that won't be an issue (except when doing a /reload in a raid). I still don't see how random people can send out mldb?

Clearing the mldb on a new ML is probably a good idea, as we're guaranteed to receive it within 15 secs (should probably also be lowered).

Also, there's no reason to clear candidates - it's really just an indentifier list with some extra info, and it will be updated whenever needed.

evil-morfar commented 6 years ago

Testing with starter accounts (no ML) always gets the groupleader as ML before mldb is received both when joining and when /reloading. I suppose the this will be the case even if there's a delay on receiving ML status, although I would really like it confirmed with ML before including such a potential breaking change.

evil-morfar commented 6 years ago

Tested with ML in a small group - seems to be working.