Rarity

Rarity

17M Downloads

Review holiday items to make sure they're using the proper holiday texture (for calendar-based notification)

rdw-software opened this issue · 5 comments

commented

In reviewing the item database, I've noticed that there's a lot of holiday items that don't appear to have their holidayTexture set (e.g., Big Love Rocket). If I understand the logic correctly, this would imply there won't be any calendar-based notification for them.

This means their holiday reminders wouldn't be working as intended. Not sure if this was changed when Blizzard moved to different reward structures (e.g., container instead of normal loot), so that's something that might need to be investigated first.

commented

I just realized after pushing the update that I didn't come across Big Love Rocket at all. Perhaps I have gone blind, but I cannot seem to find that mount. Did it get lost in the refactoring?

commented

Seems here: 0a69210

I guess because of the large diffs, the search feature of GitHub doesn't pick up on it.

commented

Can confirm it went AWOL. I must've somehow messed up copy/pasting it in. I guess that's why I'm wary of refactoring without tests :(

Looking into if there's any others I missed currently, but it's tricky to tell with just the diffs. I've tried writing some scripts but they're telling me the number of items is the same, which can't really be true, so I'm still trying to figure this one out.

commented

Done; the results can be found in a temporary debug branch (https://github.com/SacredDuckwhale/Rarity/tree/debug-db-refactoring), and the result is easily visualized in this commit: 0285129

This simulates the changes made to the database (in terms of item keys only); as we can see only the Big Love Rocket went missing, so that's a relief :D

commented

Since we can now be reasonably sure nothing else is missing, I've simply re-added the Big Love Rocket in 66e68b1.