WeakAuras

WeakAuras

206M Downloads

Spellid field using localized spellname instead of spellid

Reloe opened this issue ยท 2 comments

commented

Description

For the trigger type Player/Unit Info -> Cast if you enter a spellid into the spell Id field it is currently checking the localized spell name instead of the actual entered spellid, which can cause issues in some rather rare cases when you want to check an exact spellid but the unit has multiple casts with the same name it could be casting.
Attached aura is an example that can be used on a druid.

WeakAuras Version

WeakAuras 3.3.0

World of Warcraft Flavor

Retail (Default)

Tested with only WeakAuras

  • Yes
  • No

Lua Error

No response

Reproduction Steps

  1. Create Aura with Trigger Player/Unit Info -> Cast -> Unit:player
  2. enter a spellid for a spell you can cast that isn't the actual spellid but has the same name(I used 22373 for regrowth on druid, 8936 being the real spellid when you cast it)
  3. upon casting your spell it'll still trigger regardless of wrong spellid.

Last Good Version

idk

Screenshots

No response

Export String

!WA:2!1vv3UTrru4SYsbQfPknabIueuIureGKgsObOqrYR76AJCTDhV5xI0Mz3zS3HUE2vZmBsCURwCrU2pc(AUYpc5cUEKfpb5riIhaMz2fN0QwlzRZzMZVFZ57yRkl0Bb0cOlE4iEcK9Y9iir4hpIeetBhNYcWlpd7SMD6WXclw)CHXqAqymRvmHk8l70W1bmX4niwafKyQvY9En9NhJW2L2XTPToWHdems3Uyg)UFjlx8VTf9tW2PuIGXtWrr1qV)E4R4P(4tWur70oDiNn2RCP2UETDlbCVkLJ9G(8yMVo6hdm2zxgYft9QfdR8c0ULt96J12Nh5JhKlWloqFSoRhlVRFseSpMbOWEyEXriSVkRUQYIv1PERk7uFyknVClMCpyGGCc2ntxxc)1mgyZliokM9BfuFUwHpygfgTRQxvWWVpKJJ6yGn5Sd8HSYzMAzvqLnouKYGcC4LgSldRpWAKQBa4Uk35pEr5CJHkJuU6P(EXNK8fc8zcpZpDIz9GcVepbPh2lHHdi6Kwy4utw5bjjZB04Hqu8PzP3sL(RYoDA1DNKp0CcmveRckjWmxyxsPEDwYiNHrMdF1KSPHdYhoU0C)FKYfKo9LZoIPNbWUQdTB0SHJCoMQH02mYy4nO1W8UGksMVcJC(9FrkePae49DDNm9o97HDJykozHB1j5jF5zYlM9ZlMm)onMH2JbtgSxUqwnEIcF8JWhNJj3AOEuTgnCaE1DQ4M8zVleotZkzP3IbzcaTDOBQ92KZXlLS67kGO(QjpsGNiKH5HXrO(VwpM3tfU4JKofKRUYdOYV284j)g5DKFRCn56Vs(ql5gYzLFNCt5CYTKFVbzLpsUT8NwEg5pl)f5tow(RJZ6pqTNv1v(dYFuwEj5JFBDcnxWMQqC)qmPBO4ZTJIHOk(cyKIJTiOxAKGu0wrPcwuwViiicY5AjBUQF1cxzgOTvMJ0eLnl90NMmVI4vcPhtWim65WZc3mLGgdw9f2hs7sAT9ZgcPKEM1hBd4citu5smKJBlyyAxr47PAWQtqAkJYep96dghRiGi(aTD6bfzv7EqcTICpJ1Y9Lhipuw1VdHs4HV5XVznrOHJygMNzjqoXJPj(kY42YDlkpQOTkuIItuef1WtZeZvfhAweWjimWGX3MrVVLV6YoKUfhBoTQbuFctd7Pm8a7iY5NdzOXDyQ9qQUvXakyFECCplGXHWrI4GtY2O8PwE)7ke0e1ccw8PIW77I5I84sqimf0WzxhaagLeclCD2iEfDG1TKF7YahNgGt184)8dY(hGYtjKmcvNM)FHKBwbM8v10B16adWhP32sODTHSJ2P2A3OTwBTdJJze1eI59zu1MGAh2SHBP6J0VrKmGAmHMnGP0kETpm4LQ(iLIMUzQvHnw)rlWbBT(wRVXcN8p7)Fd

commented

For context, it was change by 4f181e3 to fix #2149

commented

Ah interesting!