Dicebar / Raven

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

cata beta error #67

Open amey1956 opened 5 months ago

amey1956 commented 5 months ago

tried to use raven in cata beta and get this error, would appreciate a fix if it isn't too complicated, spec is bm hunter. thank you

Raven/Conditions.lua:708: attempt to call global 'GetSpecialization' (a nil value) [string "@Raven/Conditions.lua"]:708: in functionUpdateConditions' [string "@Raven/Bars.lua"]:692: in function UpdateAllBarGroups' [string "@Raven/Bars.lua"]:155: in functionInitializeBars' [string "@Raven/Main.lua"]:926: in function <Raven/Main.lua:865>

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

Locals: self =

{ CheckTalent = defined @Raven/Conditions.lua:321 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:832 ReleaseBarGroup = defined @Raven/Bars.lua:723 CopyTable = defined @Raven/Profile.lua:61 ResetColorDefault = defined @Raven/Profile.lua:485 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:702 UpdateInternalCooldowns = defined @Raven/Main.lua:2146 SetExpireColor = defined @Raven/Profile.lua:690 DefaultProfile =
{ } ResetColorDefaults = defined @Raven/Profile.lua:504 RemoveMatchingTrackers = defined @Raven/Main.lua:426 Nest_CreateBar = defined @Raven/Nest.lua:1074 CheckSpec = defined @Raven/Conditions.lua:376 SetEnabledState = defined @!KalielsTracker/Libs/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:1897 ldbi =
{ } Nest_RegisterCallbacks = defined @Raven/Nest.lua:683 UnregisterAllEvents = defined @!KalielsTracker/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:164 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:1132 Nest_SetBarGroupBackdrop = defined @Raven/Nest.lua:906 DetectInternalCooldown = defined @Raven/Main.lua:2128 Nest_SetColors = defined @Raven/Nest.lua:1262 isVanilla = false InitializeSpellIDs = defined @Raven/Profile.lua:149 GetName = defined @!KalielsTracker/Libs/AceAddon-3.0/AceAddon-3.0.lua:275 InitializeBuffTooltip = defined @Raven/Main.lua:1472 name = "Raven" ColorPalette =
{ } SetConditionDefaults = defined @Raven/Conditions.lua:848 Nest_SetAlpha = defined @Raven/Nest.lua:1294 SaveBarGroupSettings = defined @Raven/Bars.lua:525 Print = defined @Chinchilla/Libs/AceConsole-3.0/AceConsole-3.0.lua:54 RegisterCooldownDefault = defined @Raven/Profile.lua:415 Nest_SetTimeColor = defined @Raven/Nest.lua:1284 Nest_DeleteBar = defined @Raven/Nest.lua:1179 ExpansionIsOrBelow = defined @Raven/Main.lua:47 UpdateSpellEffects = defined @Raven/Main.lua:1745 Debug = defined @Raven/Main.lua:224 Nest`

Dicebar commented 5 months ago

That's an impressive bug. The code it's in literally checks if the current expansion is Pandaria or more recent, and Cata definitely is not supposed to flagged as such.

That unfortunately makes it a difficult bug to fix (especially without beta access), and I'm kind of inclined to say this is a bug for Blizzard to fix.

All that said, if you comment that line out (ie -- stat.specialization = GetSpecialization()), I think Raven should work just fine for you. So that might be a workaround.

Dicebar commented 5 months ago

I'm leaving the issue open as something to look at in the future.

amey1956 commented 4 months ago

took a while to get back to read your post, but commenting out the spec line allowed me to use raven ty ;)

Dicebar commented 4 months ago

Best I can tell, Blizzard has fixed this themselves. Can you confirm this?