Originally Posted by Mitur
I only omitted things like table creation (icons table for example), and obvious stuff like event registration etc. Main logic is contained within the code I posted.
|
Not really.
Code:
if event == events["npu_added"] and not idSet:contains(unit) then
Without knowing what is going on in idSet:contains(unit) I have no idea if that logic branch is correct. I believe that part to be unnecessary but I have no way to be sure. I also don't know if you have the correct events in your table (or why you abstract them that way instead of just putting the event name there).
Code:
elseif event == events["npu_removed"] and idSet:contains(id) then
idSet:remove(id)
Same issues as above and also what does idSet:remove(id) do? What does "id" even reference and where is it set? Is it supposed to be "unit" instead or is there something going on in the code not posted.
In short post ALL of your code if you want help.