New issue report by zach.anker:
When a Dongle instance is deactivated it's passing the old instance as
self which isn't registered with lookup, so when we try to
call :UnregisterAllEvents() it's throwing the "must call from Dongle obj".
I'm not exactly sure how everything works so it's a bit of guesswork by
looking at the code, but if you change Deactivate to
local function Deactivate(self, new)
for key, val in pairs( lookup ) do
if( type( key ) == "string" ) then
self.UnregisterAllEvents( key );
lookup[ key ] = nil
end
end
end
It'll unregister all of the events of the registered modules, which is
what the goal is I think?
Issue attributes:
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
Comment #1 by zach.anker:
Of the registered modules of that Dongle instance, typo'ed
Comment #2 by jnwhiteh:
The thing is, the old instance should still be in the lookup table at that point,
which is why the logic is written as is. I'll continue to invertigate.