Grid2

Grid2

12M Downloads

Error in TWW 11.2: "Attempt to call global 'GetLootMethod' (a nil value)"

AlexFolland opened this issue ยท 10 comments

commented

After wiping saved errors with BugSack and reloading UI, I see the following message in BugSack, and Grid2 does not load fully.

2x Grid2/modules/StatusRole.lua:304: attempt to call global 'GetLootMethod' (a nil value)
[Blizzard_ExpansionLandingPage/Blizzard_ExpansionLandingPage.lua]:82: in function 'RefreshExpansionOverlay'
[Blizzard_ExpansionLandingPage/Blizzard_ExpansionLandingPage.lua]:37: in function <...pansionLandingPage/Blizzard_ExpansionLandingPage.lua:35>

Locals:
self = ExpansionLandingPage {
 overlay = <table> {
 }
 Overlay = Frame {
 }
}
newestOverlay = <table> {
}
commented

Try out 2.8.99-beta version but the addon could crash in another places, and currently i cannot access RPP 11.2 to check and fix errors: RPP Realms are displayed as incompatible for me.

commented

Yup, the fix worked just as well as my local fix, where I updated to the new function and changed the loot type check from a string to the master looter enum. Thanks!

commented

Well wait, it fixed loading, but I don't think the status can work properly any more since it's still comparing a string instead of the new enum return value.

commented

I think Grid2/modules/StatusRole.lua:305 should be changed to if method==Enum.LootMethod.Masterlooter then.

commented

I dont want to change the code yet because the code must work for classic versions too, and i want to verify in ptr that blizzard really changed the returned value from a string to a integer value.

commented

The enum's value is a number. See the following dump of Enum.LootMethod in-game.

Dump: value=Enum.LootMethod
[1]={
  Personal=5,
  Masterlooter=2,
  Group=3,
  Roundrobin=1,
  Needbeforegreed=4,
  Freeforall=0
}
commented

Why the PTR, by the way? 11.2 is live as of today.

commented

11.2 is not live in Europe yet.

commented

Try out 2.9.0 version.

commented

It works. Thanks!