Authors CurseForge

I am stuck trying to figure out how to get SleekFreeBagSlots to stop throwing an error when loading.
I know it is going to be something simple since I tend to look too hard for a problem.
This link http://www.pastey.net/142336 has the Lua as it last was; shelving my current changes just to see what I was doing wrong from the beginning.

Feel free to "whack" me since this is a little addon and should be a breeze. Right now I am just looking to stop this error:

1x sfbs-4.0.0.0\sfbs.lua:15: attempt to index global 'this' (a nil value)
sfbs-4.0.0.0\sfbs.lua:150: in function `OnEvent'
<string>:"*:OnEvent":1: in function <[string "*:OnEvent"]:1>

Locals:
event = nil
checkshown = <function> @ sfbs\sfbs.lua:19:
RegEvents = <function> @ sfbs\sfbs.lua:13:
Update = <function> @ sfbs\sfbs.lua:85:

---

I know "this" needs to be changed to "self", but I still get the error.

Thank you so much. (I will credit you with helping if you want.) You are correct, XML http://www.pastey.net/142337 will need fixing.

Current error:

1x sfbs-4.0.0.0\sfbs.lua:146: attempt to call global '_G' (a table value)
<string>:"*:OnEvent":1: in function <[string "*:OnEvent"]:1>
Locals:
self = sfbsFrame {
0 = <userdata>
}
event = "VARIABLES_LOADED"
---

I didn't write the code, but am going to try and clean it up some.
I knew I was missing something real simple. I just kept reading right over the GetGlobal and never changed it.

Back to the original error:

1x sfbs-4.0.0.0\sfbs.lua:15: attempt to index local 'self' (a nil value)
sfbs-4.0.0.0\sfbs.lua:150: in function `OnEvent'
<string>:"*:OnEvent":1: in function <[string "*:OnEvent"]:1>
Locals:
self = nil
event = nil
checkshown = <function> @ sfbs\sfbs.lua:19:
RegEvents = <function> @ sfbs\sfbs.lua:13:
Update = <function> @ sfbs\sfbs.lua:85:
---

As noted, the code is a mess, and I guess that's why I can't figure this out.
New Lua http://www.pastey.net/142339 and XML http://www.pastey.net/142337


And thanks for the help so far.

Considering this is nowhere near sleek I'd suggest finding a replacement.

Anyway, from the xml file change the onevent code to

<OnEvent> 
    sfbs.OnEvent(self, event, ...); 
</OnEvent> 

and for the lua file change the relevant function to

function sfbs.OnEvent(self, event, ...) 
    if not sfbsOPT then sfbsOPT={skipamm = true}; end 
    sfbs.bag = {} 
    for i = 0,4 do 
        sfbs.bag[i] = _G[("sfbsBag"..i.."Text")] 
    end 
    checkshown() 
    RegEvents(self) 
    self:UnregisterEvent("VARIABLES_LOADED") 
    sfbs.OnEvent = Update 
end  

Drycoded but that addon hurts my eyes.

It's more convience for me then necessary, but I like seeing at a glance how much space I have available. The downloads from the three main addon sites would indicate that a lot of people like this addon and if it is possible to save it, then I'll try (with a lot of help from my Friends).
The only other thing close I've seen are on Titan or Fubar. The other current bag addons I've seen are "one bag" types. I like the Blizz bags fine; not into a huge thing on my screen.

Anyway, what ever the issue is, it still is rearing its ugly error, though it is a bit different this time:

1x sfbs-4.0.0.0\sfbs.lua:15: attempt to index local 'self' (a nil value)
sfbs-4.0.0.0\sfbs.lua:150: in function `OnEvent'
<string>:"*:OnEvent":1: in function <[string "*:OnEvent"]:1>
Locals:
self = sfbsFrame {
0 = <userdata>
}
event = "VARIABLES_LOADED"
checkshown = <function> @ sfbs\sfbs.lua:19:
RegEvents = <function> @ sfbs\sfbs.lua:13:
Update = <function> @ sfbs\sfbs.lua:85:
---

Check my post above, specifically the lua code section.
I updated with the missing param.
Just copy-paste it and replace the function in your file again.

That appears to have done it.
Thanks to all 3 of you. I'll tag credit if any of you want, I appreciate it.
I always have problems seeing the small things, and the harder I try, the more I miss.

Quote from Dridzt

Drycoded but that addon hurts my eyes.

Is there a simple sample somewheres I can look at to clean this one up before submitting it to the download sites?