Bagnon icon indicating copy to clipboard operation
Bagnon copied to clipboard

Error when attempting to set Character Specific Settings

Open b-morgan opened this issue 1 year ago • 3 comments

Which software were you running?

  • Addon version name: Bagnon 10.1.11
  • Client used: Wrath 3.4.2

Have you read the changelog? Yes

Please describe the bug. Error when attempting to set Character Specific Settings. Note that even though the option got set, it doesn't work.

Please describe how to reproduce it.

  1. Open Bagnon settings
  2. Click on Character Specific Settings
  3. See error

Any Screenshots? N/A

Error Logs are Important! 2x BagBrother/core/api/owners.lua:124: attempt to index global 'owner' (a nil value) [string "@BagBrother/core/api/owners.lua"]:124: in function SetProfile' [string "@BagBrother/config/panels/generalOptions.lua"]:47: in function SetProfile' [string "@BagBrother/config/panels/generalOptions.lua"]:32: in function ToggleGlobals' [string "@BagBrother/config/panels/generalOptions.lua"]:27: in function method' [string "@Scrap/libs/Sushi-3.1-5/classes/core/Callable.lua"]:55: in function `FireCalls' [string "@BagBrother/libs/Sushi-3.1-5/classes/buttons/Clickable.lua"]:39: in function <...Brother/libs/Sushi-3.1/classes/buttons/Clickable.lua:34>

Locals: self =

{ profile =
{ } name = "Gracob" address = "Gracob-Mankrik" id = "Gracob" offline = false cache =
{ } realm = "Mankrik" } profile =
{ vault =
{ } inventory =
{ } guild =
{ } bank =
{ } } (*temporary) = defined @BagBrother/core/api/settings.lua:155 (*temporary) =
{ UnregisterSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:78 GetProfile = defined @BagBrother/core/api/settings.lua:155 SetProfile = defined @BagBrother/core/api/settings.lua:150 RegisterSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:74 NewModule = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:68 UnregisterAllEvents = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:164 RegisterEvent = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:80 UnregisterAllMessages = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:164 OnEnable = defined @BagBrother/core/api/settings.lua:97 RegisterMessage = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:80 SendSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:82 UnregisterMessage = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:143 SendMessage = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:49 UnregisterEvent = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:143 Tag = "BAGNON_" } (*temporary) = nil (*temporary) = "Gracob" (*temporary) =
{ vault =
{ } inventory =
{ } guild =
{ } bank =
{ } } (*temporary) = "attempt to index global 'owner' (a nil value)" Addon =
{ IsKeyring = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:420 FrameTemplate = "BackdropTemplate" IsRetail = false GetOwnerID = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:300 InventoryBags =
{ } Currency =
{ } InBank = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:397 ItemGroup =
{ } GeneralOptions = Frame { } ContainerItemGroup =
{ } player =
{ } RestoreItemData = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:334 Frame =
{ } OptionsToggle =
{ } Bag =
{ } IsBank = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:424 RegisterMessage = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:80 UnregisterMessage = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:143 IsBagCached = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:321 CurrencyTooltipCounts =
{ } Base =
{ } UnregisterSignal = defined @BagBrother/libs/WildAddon-1.0/WildAddon-1.0.lua:78 GetOwnerAddress = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:305 RegisterEvent = defined @Accountant_Classic/Libs/CallbackHandler-1.0/CallbackHandler-1.0.lua:80 sets =
{ } OnEnable = defined @BagBrother/core/core.lua:36 Name = "Bagnon" IterateOwners = defined @BagBrother/libs/LibItemCache-2.0/LibItemCache-2.0.lua:135 InGuild = defined @Int

b-morgan avatar Sep 27 '23 12:09 b-morgan

I'm also seeing this issue, fresh install so my error text isn't as large

Message: Interface/AddOns/BagBrother/core/api/owners.lua:124: attempt to index global 'owner' (a nil value) Time: Sat Sep 30 10:47:04 2023 Count: 1 Stack: Interface/AddOns/BagBrother/core/api/owners.lua:124: attempt to index global 'owner' (a nil value) [string "=(tail call)"]: ? [string "=[C]"]: ? [string "@Interface/AddOns/BagBrother/core/api/owners.lua"]:124: in function SetProfile' [string "@Interface/AddOns/BagBrother/config/panels/generalOptions.lua"]:47: in function SetProfile' [string "@Interface/AddOns/BagBrother/config/panels/generalOptions.lua"]:32: in function ToggleGlobals' [string "@Interface/AddOns/BagBrother/config/panels/generalOptions.lua"]:27: in function method' [string "@Interface/AddOns/Scrap/libs/Sushi-3.1/classes/core/Callable.lua"]:55: in function FireCalls' [string "@Interface/AddOns/BagBrother/libs/Sushi-3.1/classes/buttons/Clickable.lua"]:39: in function <...Brother/libs/Sushi-3.1/classes/buttons/Clickable.lua:34>

Locals: (*temporary) = "LeftButton" (*temporary) = false (*temporary) = defined =[C]:-1`

DudicusMaximus avatar Sep 30 '23 16:09 DudicusMaximus

Changing line 124 to this

self.profile = Addon.Settings:GetProfile(self.realm, self.id)

appears to allow setting on an single character and retain said settings. Do not know if this is the proper fix but it works..

Thrumbar avatar Sep 30 '23 23:09 Thrumbar

Changing line 124 to this

self.profile = Addon.Settings:GetProfile(self.realm, self.id)

appears to allow setting on an single character and retain said settings. Do not know if this is the proper fix but it works..

Seems to have worked for this issue, thanks!

DudicusMaximus avatar Oct 01 '23 04:10 DudicusMaximus