Luxocracy / NeatPlates

NeatPlates is a continuation of the original Tidy Plates in an attempt to keep it update date in the absence of its authors.
https://wow.curseforge.com/projects/neat-plates
80 stars 20 forks source link

Classic/Retail Compatibility issue with _G["Settings"] #387

Closed orionshock closed 1 week ago

orionshock commented 1 year ago

What version of the game you are experiencing the issue with: Wrath Classic NeatPlates-v427 -- Latest from Curseforge.

Describe the bug This seems to be part of a Classic/Retail compatibility check. NeatPlatesHub checking for the Blizzard Table _G["Settings"] and it's crashing / conflicting with ProfessionMaster addon. This happens if Profession Master loads first then NeatPlates has a bit of a conniption fit.

To Reproduce Steps to reproduce the behavior:

  1. Profession Master Loads first
  2. NeatPlates loads afterwards
  3. Lots of Errors starting with
1x Interface\AddOns\NeatPlatesHub\Templates.lua:1350: attempt to call field 'GetCategory' (a nil value)
[string "@Interface\AddOns\NeatPlatesHub\Templates.lua"]:1350: in function <Interface\AddOns\NeatPlatesHub\Templates.lua:1033>
[string "@Interface\AddOns\NeatPlatesHub\Hub.lua"]:797: in function <Interface\AddOns\NeatPlatesHub\Hub.lua:781>
[string "@Interface\AddOns\NeatPlatesHub\Hub.lua"]:817: in function <Interface\AddOns\NeatPlatesHub\Hub.lua:815>
[string "@Interface\AddOns\NeatPlatesHub\Hub.lua"]:979: in function <Interface\AddOns\NeatPlatesHub\Hub.lua:963>

I have also raised this issue with ProfessionMaster Addon https://discord.com/channels/831203449675841567/938845705349529640/1053850331584397343

Luxocracy commented 1 year ago

Ah, I'm assuming their solution to the compatability was to create the "Settings" global that doesn't normally exist outside of dragonflighr. I would generally consider this bad practice, but to be fair I should probably also do a version check and not entirely rely on if this global exists.

I'll hopefully have some time today to check on this issue.

orionshock commented 1 year ago

Looking into this further, ProfessionMaster is straight up using "Settings" as a saved variable . . .

github-actions[bot] commented 1 month ago

This issue is stale because it has been open for 90 days with no activity. If you feel the issue is still relevant, please leave a comment to prevent the issue from being closed in 30 days