Dicebar / Raven

Raven add-on for World of Warcraft
9 stars 3 forks source link

LUA error on Wrath PTR #45

Closed sausje closed 1 year ago

sausje commented 1 year ago

Getting the following error on PTR

1x Raven/Main.lua:1033: attempt to call global 'GetContainerItemID' (a nil value) [string "@Raven/Main.lua"]:1033: in function `BAG_UPDATE' [string "@Raven/Main.lua"]:850: in function <Raven/Main.lua:840>

[string "@Ace3/AceAddon-3.0-13/AceAddon-3.0.lua"]:66: in function <Ace3/AceAddon-3.0/AceAddon-3.0.lua:61> [string "@Ace3/AceAddon-3.0-13/AceAddon-3.0.lua"]:523: in function `EnableAddon' [string "@Ace3/AceAddon-3.0-13/AceAddon-3.0.lua"]:626: in function <Ace3/AceAddon-3.0/AceAddon-3.0.lua:611>

Locals: self =

{ CheckTalent = defined @Raven/Conditions.lua:398 testOrder =
{ } HexColor = defined @Raven/Profile.lua:43 UpdateInCombatBar = defined @Raven/Cancel.lua:66 modules =
{ } Nest_SetShine = defined @Raven/Nest.lua:1303 GetValueFormat = defined @Raven/Values.lua:833 ReleaseBarGroup = defined @Raven/Bars.lua:723 CopyTable = defined @Raven/Profile.lua:61 ResetColorDefault = defined @Raven/Profile.lua:452 Nest_SetBarGroupLock = defined @Raven/Nest.lua:973 Nest_FormatTime = defined @Raven/Nest.lua:1873 SetInCombatBar = defined @Raven/Cancel.lua:203 LocalSpellNames =
{ } GetExpireColor = defined @Raven/Profile.lua:669 UpdateInternalCooldowns = defined @Raven/Main.lua:2119 SetExpireColor = defined @Raven/Profile.lua:657 DefaultProfile =
{ } ResetColorDefaults = defined @Raven/Profile.lua:471 RemoveMatchingTrackers = defined @Raven/Main.lua:401 Nest_CreateBar = defined @Raven/Nest.lua:1074 CheckSpec = defined @Raven/Conditions.lua:453 SetEnabledState = defined @Ace3/AceAddon-3.0/AceAddon-3.0.lua:420 FinalizeBars = defined @Raven/Bars.lua:161 mountSpells =
{ } Nest_SetLabelColor = defined @Raven/Nest.lua:1279 LoadCustomBars = defined @Raven/Bars.lua:528 Nest_DeleteAllBars = defined @Raven/Nest.lua:1243 Nest_AdjustColor = defined @Raven/Nest.lua:1344 UpdateTrackers = defined @Raven/Main.lua:1870 ldbi =
{ } Nest_RegisterCallbacks = defined @Raven/Nest.lua:683 UnregisterAllEvents = defined @Ace3/CallbackHandler-1.0/CallbackHandler-1.0.lua:169 InitializeBarGroup = defined @Raven/Bars.lua:494 Nest_SetBarGroupBorder = defined @Raven/Nest.lua:898 Nest_GetBarGroups = defined @Raven/Nest.lua:796 HideBars = defined @Raven/Bars.lua:175 OptionsPanel = defined @Raven/Main.lua:1105 Nest_SetBarGroupBackdrop = defined @Raven/Nest.lua:906 DetectInternalCooldown = defined @Raven/Main.lua:2101 Nest_SetColors = defined @Raven/Nest.lua:1262 warlockCreatures =
{ } InitializeSpellIDs = defined @Raven/Profile.lua:141 GetName = defined @Ace3/AceAddon-3.0/AceAddon-3.0.lua:275 InitializeBuffTooltip = defined @Raven/Main.lua:1445 name = "Raven" ColorPalette =
{ } SetConditionDefaults = defined @Raven/Conditions.lua:937 Nest_SetAlpha = defined @Raven/Nest.lua:1294 SaveBarGroupSettings = defined @Raven/Bars.lua:525 Print = defined @Ace3/AceConsole-3.0/AceConsole-3.0.lua:50 Nest_SetTimeColor = defined @Raven/Nest.lua:1284 Nest_DeleteBar = defined @Raven/Nest.lua:1179 ExpansionIsOrBelow = defined @Raven/Main.lua:43 UpdateSpellEffects = defined @Raven/Main.lua:1718 Debug = defined @Raven/Main.lua:206 Nest_IsTimer = defined @Raven/Nest.lua:1256 Nest_GetBarGroup = defined @Raven/

Dicebar commented 1 year ago

Hoi sausje, thanks for the report!

@dorana: Does the PTR count as "main line" for the sake of determining if the API is modern or legacy?

sausje commented 1 year ago

Hoi sausje, thanks for the report!

@dorana: Does the PTR count as "main line" for the sake of determining if the API is modern or legacy?

Ah apologies, the Wrath PTR

dorana commented 1 year ago

Yes it should, but I see that there is a version missmatch, Hold on and I will see if I can adjust

Dicebar commented 1 year ago

Thanks!

dorana commented 1 year ago

I have pushed r629 this checks for version greater than rather than exact version, PTR moved from 3.4.1 -> 3.4.2 and this caused the PTR to attempt to load the old API again

sausje commented 1 year ago

Working flawless now, at least for the purpose that I use it.