WeakAuras

WeakAuras

206M Downloads

BonusID matching is matching on itemid

thenoid opened this issue ยท 0 comments

commented

Description

i have a weapon, the devouring cold, which has itemid 187056 (https://www.wowhead.com/item=187056/) I also have a runecrafter lego, the magistrate's judgement, which has a bonusid of 7056 (https://www.wowhead.com/spell=337681/the-magistrates-judgment). Notice that 7056 is a substring of 187056.

When creating a weakaura to warn me if I don't have that legendary equiped, it will give a false positive based on my sword being equiped.

it appears this is a bug in this function WeakAuras.CheckForItemBonusId:

WeakAuras.CheckForItemBonusId = function(ids)

and might have been a regression introduced when trying to fix this bug #3223

the item string for the devouring cold is item:187056:6229:::::::60:66::6:5:7187:6652:1498:6646:7725:1:28:2106:::::

WeakAuras Version

3.73

World of Warcraft Flavor

Retail (Default)

Tested with only WeakAuras

  • Yes
  • No

Lua Error

No response

Reproduction Steps

  1. Create a WA looking for bonusid7056
  2. equip item 187056
  3. watch the WA misfire.

Last Good Version

unknown

Screenshots

using the attached WA:

  • screenshot 1 shows no lego equipped, and a different sword, and the WA not popping
  • screenshot 2 shows no lego equipped, devouring cold equipped, and the WA popping.

WoWScrnShot_110421_024827
WoWScrnShot_110421_024720

Export String

!WA:2!1rvZUTTrq4iOdfGOi)Odgji9GqksbccGrAnI91QvLevUYwku01oihOxXDO4gqU7MDxkBLJc9qp7hbDUN0JqEcie6tGFe8Hc0BzwsLc3IAEzNzW8738nSvqNIoSoSF)5AzPnNla9OtIgo4y)WezUuFyB8t9sg3OYPlIGlTXPsDb1gRIT8ciMTqql4jX2mnyYK5Sf6fJstnGT1AQijtQhl5c70((hh5hQ6E3zQrRL6HjLgRSW5Xjkg1cHWCqyVMwIwPwEYPCMnJ0dv10elxkm7hASuT1BAkxWnzEe8X6T0Q5ZMbAZ9)o9wX)Aj3cfhtlGwKs0NPUEb0xBkNwxKjLPP8lxh3V3KO4jr9cJ2mdqmHNmjtEXiX1M6N(sCqLxiupQ0aX)lpo)totFPkNhIOdy8U5lgjsrPza78MiZbmwgvVyqIuC(sJcYZhWmE)t)mwdy)eozS)WHvEQDgGPOBDo6oG11)dLCLcyRVvMjh8QxVpXUqbeN1vmykotrOU(N9hoo4KHxvk2IgEQh5aW5quJ(rsg8h3t90)3DuJqOBvXUb3OGwqZ)veDXfW7wEHuZovtvlpDRajvkSQhgO5FS7BkPmn1s7gfTcNSqyMBP5rYLuwWulnhN0DclkZTCpcccj7u9AVWKCQX4Kig(hbNWsxoNGkpzJjJIlGZAyATxPRtPBkjwSRxHWjJxtn8uF7Dt5uAiH7gG2xrfCh5sk2V6(bFcOgyIvdIz2SVQ65eHuaByL6ApIDOR2aUIyw6C1v4QEKckxeu9Jyav9Qiv9R(j89b)xl6l3EFOFps05Plid9dIwzLjZBqZh3k(VFgNfE0HJLjBN032eZ3CV1PAKrH9gINTVP5clWzY1dtN0p03)4PyNLYN59dLC26x4FY8Fr)9rPVy2g8ecDFKQbzwFlGzZrh2LB6cogfsOAkA)6)a0QvR2RqwiWQp8(TVEnx0aJyw8UYa5PnN4KrrrJoQJjCVDpy396m)pp7Z)