View Single Post
09-19-19, 11:09 PM   #14
Sythalin
Curse staff
 
Sythalin's Avatar
AddOn Author - Click to view addons
Join Date: Aug 2006
Posts: 680
Originally Posted by erispope View Post
I think I found the issue - you are using info.check in the new code, but info.checked in the old code (and checked is what Blizzard code is using).
Well I'll be damned. This is why I asked if someone would notice something I missed, which I clearly did.

Just for the knowledge to anyone else who may come across this, I reverted the changes made from working with Fizzlemizz to test this (no table.wipe, functions placed back after constructors) and made the change to set.checked. Reloaded and it works as intended.

So both solutions worked, just in different ways (but now adds the mystery on why the Fizzlemizz changes still allowed proper functionality with self.check still in there...)

As for the reformat suggestion, I never gave much thought to it. As I said, I've been out of coding for a few years now, so it didn't strike me that that particular format would cause unnecessary bloat. I have a few other spots I can apply this to as well. Thanks for pointing it out.
  Reply With Quote