Dragonflight UI Classic

Dragonflight UI Classic

502k Downloads

[BUG] Error with Minimap Alert addon

brutzel opened this issue ยท 7 comments

commented

Describe the bug
The addon Minimap Alert constantly scans for herbs and mining nodes by resizing the minimap and moving it along the cursor position and back to its origin to trigger minimap tooltips. This was compatible with DragonflightUI a while ago but then it broke. When I enter combat I get the lua error below.

To reproduce

  • Install both addons
  • run /minimapalert
  • Hit Start
  • Enter combat
  • See error below

I will make another donation if this gets fixed ๐Ÿ˜Ž

Error Log

Message: Minimap:SetPoint(): Usage: ("point" [, region or nil] [, "relativePoint"] [, offsetX, offsetY]
Time: Sun Dec 15 02:42:22 2024
Count: 1
Stack: Minimap:SetPoint(): Usage: ("point" [, region or nil] [, "relativePoint"] [, offsetX, offsetY]
[string "@Interface/AddOns/Minimap_Alert/Minimap_Alert.lua"]:239: in function <Interface/AddOns/Minimap_Alert/Minimap_Alert.lua:234>
[string "@Interface/AddOns/Minimap_Alert/Minimap_Alert.lua"]:536: in function `?'
[string "@Interface/AddOns/Minimap_Alert/Minimap_Alert.lua"]:20: in function <Interface/AddOns/Minimap_Alert/Minimap_Alert.lua:17>
[string "@Interface/AddOns/Minimap_Alert/Minimap_Alert.lua"]:448: in function <Interface/AddOns/Minimap_Alert/Minimap_Alert.lua:412>

Versions (please complete the following information):

  • WoW: Era
  • Addon 0.19.13 with Minimap Alert 26

Additional context
Add any other context about the problem here.

commented

Thanks! Feel free to open another issue for anything you notice, or if you have suggestions.

Enjoy the holidays! โค๐ŸŽ„

commented

That fix came fast, thanks! I went back checking older versions with CurseForge client and the first broken version with those two addons seems to be 0.16.2 when entering combat. It is another error message though. That version is hardly compatible with latest era client so I don't know if the root cause is there.

Message: ...face/AddOns/DragonflightUI/Mixin/Statusbar.mixin.lua:182: attempt to index local 'state' (a nil value)
Time: Mon Dec 16 10:49:12 2024
Count: 2
Stack: ...face/AddOns/DragonflightUI/Mixin/Statusbar.mixin.lua:182: attempt to index local 'state' (a nil value)
[string "@Interface/AddOns/DragonflightUI/Mixin/Statusbar.mixin.lua"]:182: in function `Update'
[string "@Interface/AddOns/DragonflightUI/Mixin/Statusbar.mixin.lua"]:24: in function <...face/AddOns/DragonflightUI/Mixin/Statusbar.mixin.lua:23>

Anyway, looking forward to a new release with that option!

commented

When did it work? I have not really touched the minimap in the latest updates. The last big change was like 2 months ago when I introduced the state handler to hide frames out of combat etc.

commented

It was the state handler; I added those to hide frames based on macro conditionals, even in combat, but they make the frames secure.
Your addon moves the minimap around a lot, even in combat, which seems fine, but not with secure frames attached, so some calls get protected and will error out.

My solution is not the best as I dont really like too many options, thats why I kind of buried it there on the end ๐Ÿ˜

commented

Should be fixed with 8996da6

You need to disable this setting inside minimap and buff settings:
image
(and /reload)

I will push out the update later, might do 1-2 more fixes for this version.

commented

v0.19.14
You can get it here, or wait for curseforge (they take like 15-30min to approve)

commented

Thank you very much, 0.19.14 fixed it (With "Use State Handler" option disabled) ๐Ÿ˜Š Donation incoming ๐Ÿ’ต๐Ÿ’ต๐Ÿ’ต

Just FYI it still says Interface action failed because of an AddOn when scanning+entering combat but no Lua error. But MinimapAlert is working fine now so I don't really care.