Zahariel1942 / D2Stats

16 stars 7 forks source link

Notifier delay while hiding items #13

Open azadix opened 1 month ago

azadix commented 1 month ago

Hello,

recently I was testing notifier rules with large amount of rare mods and encountered small problem with this item:

image

When notifier has only those 2 rules it corretly shows item on ground and notifies about the Ice Lance affix: hide sacred normal magic rare rare {[2-9][0-9] to Ice Lance}

Problem occurs when i add those 2 mods to a very long notifier. Hide at the beginning of the file and notify later down the line. In that situation the notifier doesn't fire, belt is shown on the ground for a split second and is immediatly hidden after. Other rare items that are not matching any stat groups get hidden corretly. Here's the notifier in which the problem occurs if that helps: https://pastebin.com/9mUyP0ai

Version: 4.0.3

azadix commented 1 month ago

Additional testing showed that the bug only shows up when adding multiple lines that match specific rarity eg. magic rare: rare {[2-9][0-9] to Ice Lance} rare {(1[5-9]|[2-9][0-9]) to Summon Sinner} rare {Unseelie} rare {Guided Chain} rare {Mana on Striking}

While there is more that 1 line containing: RATITY {STAT_MATCH} eg. rare {Life} AND rare {Skills} it breaks the drop filter. You have to condense it into 1 line then it starts working again OR include "" before every line