WeakAuras

WeakAuras

Addons
206M Downloads

Empty strings in Player Level and Player Effective Level load options cause breaking error.

nullKomplex opened this issue ยท 0 comments

commented

Describe the bug

When using either the Player Level or Player Effective Level load options, when you select a numerical comparator and set the value to an empty string an error occurs. This error prevents WA from loading on a reload.

Note that in this scenario an empty string is different than the default blank value (which is simply no data). For an empty string you must select the field as to input a value and press enter to confirm.

Do you have an error log of what happened?
When setting the value:

1x [string "--[==[ Error in 'New':'load' ]==] return fu..."]:3: attempt to compare number with string
[string "--[==[ Error in 'New':'load' ]==] return function(event, combat, encounter, warmode, petbattle, vehicle, vehicleUi, ingroup, player, realm, class, spec, class_and_spec, covenant, race, faction, level, effectiveLevel, zone, zoneId, zonegroupId, encounterid, size, difficulty, instance_type, role, affixes)
local never = false
if((true) and (true) and (level<[[]])) then
return true else return false end end"]:3: in function `loadFunc'
[string "@WeakAuras\WeakAuras-@[email protected]"]:1462: in function <WeakAuras\WeakAuras.lua:1363>
[string "@WeakAuras\WeakAuras-@[email protected]"]:1518: in function `ScanForLoads'
[string "@WeakAurasOptions\LoadOptions.lua"]:374: in function <WeakAurasOptions\LoadOptions.lua:368>
[string "=[C]"]: ?
[string "@WeakAuras\Libs\AceConfig-3.0-3\AceConfigDialog-3.0\AceConfigDialog-3.0-79.lua"]:51: in function <...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:49>
[string "@WeakAuras\Libs\AceConfig-3.0-3\AceConfigDialog-3.0\AceConfigDialog-3.0-79.lua"]:843: in function <...nfig-3.0\AceConfigDialog-3.0\AceConfigDialog-3.0.lua:664>
[string "=[C]"]: ?
[string "@WeakAuras\Libs\AceGUI-3.0\AceGUI-3.0-41.lua"]:72: in function <...faceWeakAuras\Libs\AceGUI-3.0\AceGUI-3.0.lua:70>
[string "@WeakAuras\Libs\AceGUI-3.0\AceGUI-3.0-41.lua"]:306: in function `Fire'
[string "@WeakAuras\Libs\AceGUI-3.0-41\widgets\AceGUIWidget-EditBox.lua"]:74: in function <...ras\Libs\AceGUI-3.0\widgets\AceGUIWidget-EditBox.lua:71>

When reloading:

1x [string "--[==[ Error in 'New':'load' ]==] return fu..."]:3: attempt to compare number with string
[string "--[==[ Error in 'New':'load' ]==] return function(event, combat, encounter, warmode, petbattle, vehicle, vehicleUi, ingroup, player, realm, class, spec, class_and_spec, covenant, race, faction, level, effectiveLevel, zone, zoneId, zonegroupId, encounterid, size, difficulty, instance_type, role, affixes)
local never = false
if((true) and (true) and (level<[[]])) then
return true else return false end end"]:3: in function `loadFunc'
[string "@WeakAuras\WeakAuras-@[email protected]"]:1462: in function <WeakAuras\WeakAuras.lua:1363>
[string "@WeakAuras\WeakAuras.lua"]:1646: in function `Resume'
[string "@WeakAuras\WeakAuras.lua"]:1023: in function <WeakAuras\WeakAuras.lua:981>

[string "@WeakAuras\WeakAuras-@[email protected]"]:1131: in function <WeakAuras\WeakAuras.lua:1082>

To Reproduce

Steps to reproduce the behavior:

  1. Create new aura.
  2. In the Load tab enable Player Level
  3. Set Player Level's comparator option to <
  4. Select the number field.
  5. Press enter.
  6. Observe error.
  7. Reload and observe breaking error.

Alternatively a sample import has been included below.

Did you try having WeakAuras as the only enabled addon and everything else (especially something like ElvUI) disabled?
Tested with only the official WA addons and bugsack/grabber enabled.

Which version of WeakAuras are you using?
9004d32

Are you on World of Warcraft Classic or Retail?

  • Classic
  • Retail

Was it working in a previous version? If yes, which was the last good one?
Unknown.

Additional Info

!WA:2!1nvZUnTsu4gYcqw9IkbPQlcwukCz1DvwurHUPoyR2IBsQJ7pWgZyphNmvoZmAMXPjDzecX68iK1Skpc3NaRkEc6JqFc4mXrxHqG34Z5mNZ8DoFFNPMFJHnOnOF9LkrHjNXbvNtJcoSTxyQixOoQo(TGY0YCYKiySz7)rQM0jltdMAYN)theNjudjMyzSHneIPt4KHS0bYnsl0gXqBkNkPedecJaU5wsbgLyyPNZOMbU7JUksQHj46Dc1gIY4KKX4m9ahx8NXzQrX63hu6h(k1kZVmNcjfzzrtKG6aVGU(Ng4AqNqsHI0CQwc55hs1o3QlswcBpmB24fXT2VxuCVO9dJClWlpXofGke7zq7u(GKdasUzW)xwxfGLf2RRxqWSc(k4DKpY2XJGOk)Jfu4BRjF6VLvQmcTKd9og3akoj)mCCWj(KPxju0Zve50Zxz4Mj4g5g(k21BDsbHQigYwrrZXoke6Bzjh3CbHo62CSdZJfidqqkSXEZk0q8YGFkXqYXUFZWHf5gMJlYhPBw(VoHlp(EHP5eT2gWvZUgSgtTW2dDEYn6beQ4QlQ066Zvlr1Y0UgCWMNk4u2s5Yr(I)8EGubPm7mwFgHZScUGVt56()hq0qpJc49ndUF5JD5coCdfLnBgXwruPbli6P2uTaxUR7qcJ7x(ASGYDlFt5Bl3d))x)Ae14vBOQlXLpw2e3ap)O5gr6Okc)VRfpBBgTzB4QeeJmwFNfzku8Xocj663r4PdekFBilYj9Af651(gCPfd3rwn3nly0fJF)Ka)pE0zjnVCfL9HkWF2AlQULUcuUtA51oYlCrvkTw(2QwTA1NJ7vaD5BGpV(cgVI9WR3zMgYZQQ1TtuuNJBOB(oyuJrF)IF8d

There may exist imports on wago of this invalid data, so that may be something to consider when fixing.