Mageblood Potion and Nightfin Soup are treated as if only one of them can be active at a time
pajlada opened this issue · 4 comments
When popping a Mageblood Potion, the Nightfin Soup icon is removed.
I believe this has to do with Nightfin Soup causing the Mana Regeneration buff to activate, and Mageblood Potion also causing a different Mana Regeneration buff.
Issue seems to be two-fold:
- If the user has the Nightfin Soup buff active, the BuffCheck3.lua:783 check is causing Mageblood Potion to not show up because the buff name is identical
- If the user has the Mageblood Option buff active, the BuffCheck3.lua:772 check is causing Nightfin Soup to not show up because
Mana Regeneration
is listed as a food buff
Thanks for the report, I knew this was happening but never made time to fix it. The correct fix is to switch everything to spell ids instead of checking the name of the buff but I don't want to break something else atm.
For now, I added a check in IsBuffPresent and another in IsFoodBuffPresent. These will double check the spell ids if the name of the buff is "Mana Regeneration".
From my brief testing with Nightfin Soup, Mageblood Potion is staying on the bar, and Nightfin's 5min/2min warnings are working correctly. If I remember I'll test Magebloods interactions when I next raid on my druid (saturdays).
Pushing the quick fix, I'll update curseforge as well after double checking mageblood. Thanks again for the help!
Awesome! If you haven't gotten a release out by Sunday I'll try the git version out myself and report back ❤️ Thank you!
The quick fix appears to be working for me, I pushed it to this repo and curseforge. I think the overwolf client should already have the new update. Let me know if theres any other issues. Feel free to close this if its working on your end, thanks!
@Ghostjs123 Got to try it out today, works great! Thanks a lot for the quick fix