WowRarity / Rarity

Collectibles and statistics tracking addon for World of Warcraft
https://www.curseforge.com/wow/addons/rarity
GNU General Public License v2.0
55 stars 35 forks source link

Class-restricted items erroneously trigger attempts for Dracthyr #774

Open rdw-software opened 2 hours ago

rdw-software commented 2 hours ago

Source: Discord

Toy - Skull of Corruption drop of Gul'dan in Nighthold. Rarity is tracking attempts with Dracthyr. As I understand it the only class this drops for is DH.

This is due to the nature of disableForClass being opt-out. Whenever a new class is added, all items with this setting have to be updated. Thankfully, there's only a few of them in the database, but this would also affect custom items. Maybe there's a way to avoid having to manually update these in the future, which is likely to be forgotten. If not, could at least add some unit tests or other reminders that make sure a mismatch between the list of supported classes and disableForClass settings doesn't survive the CI pipeline.

Affected items:

rdw-software commented 2 hours ago

The immediate problem has been fixed via #775. I'll leave this open because future desyncs are still possible without tests.