This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 31478 - Problem with PROP_COOKIES when module is disabled
Summary: Problem with PROP_COOKIES when module is disabled
Status: CLOSED DUPLICATE of bug 30244
Alias: None
Product: platform
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 3.x
Hardware: PC Linux
: P2 blocker (vote)
Assignee: David Konecny
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-02-26 10:37 UTC by mslama
Modified: 2008-12-22 15:53 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description mslama 2003-02-26 10:37:05 UTC
When I disable module (projecs in this case) I
should clean cache in PersistenceManager. I listen
to InstanceDataObject as I was told by jpokorsky.
See issue #25236.

1.Fire PROP_COOKIE when module is disabled. It works.
2.When I receive PROP_COOKIE in my listener
InstanceCookie should be
null for components owned by disabled module. It
is NOT so.

This condition was used to detect that module
owning given
TopComponent was disabled so PersistenceManager
should remove refrence
to given TopComponent from its cache. I will file
new issue for it.

Not sure if assign to settings or data systems.
Comment 1 mslama 2003-02-26 10:44:16 UTC
I checked today dev build 030226. It worked before as issue #25236 was
fixed. I do not know when or why it stopped to work.
Comment 2 David Konecny 2003-02-27 13:22:17 UTC
Honza, what's the problem here? I would apreciate your suggestion what
to do, what to check. Issue 31486 also talks about firing PROP_COOKIE.
Any relation?

For the moment setting TM to 3.5
Comment 3 David Konecny 2003-02-28 13:16:30 UTC
Reminder for myself: Honza reccommend me to look at the
SerialDataConvertor and recent changes in listening on installed
modules done by Radek. The change might be related to this issue.
Comment 4 Vitezslav Stejskal 2003-03-03 12:12:48 UTC
Also consider that Project tab is now sigleton (it wasn't before it
has been moved from core to project module). I found that singletons
don't get GCed when module is uninstalled. I don't know whether it is
known problem. Actually I am not sure whether this is the reason or
consequence of this bug :-(.

Perhaps Jesse could know more about singletons in modules and their
impact on the module's lifecycle.
Comment 5 David Konecny 2003-03-03 17:29:47 UTC
I asked Jesse the same question and he explained me that uninstalling
module causes garbage collecting of module's classloader and
everything it references what includes also statically referenced
singleton in the module. This of course works only if there is not
other reference to the module from the outside.
Comment 6 Peter Zavadsky 2003-03-05 12:35:20 UTC
Raising priority, since it blocks issue #30244 which is P2.
Comment 7 David Konecny 2003-03-06 09:41:38 UTC
Closing as duplicate. There is problem in project's definition of
.settings file. The "module" tag was missing in settings file and so
settings system could not clean up this project setting during the
module uninstall.

*** This issue has been marked as a duplicate of 30244 ***
Comment 8 mslama 2003-04-29 09:14:11 UTC
I must reopen. It does NOT work when settings file is present at local
layer:
1.Start IDE with empty user dir.
2.Exit IDE. settings files of all loaded TopComponents are saved to
local layer (including file project.settings).
3.Start IDE.
4.Disable projects module.
5.No event PROP_COOKIE with null instance cookie for project IDO will
come.

It works when project.settings file is only at module layer:
1.Start IDE with empty user dir.
2.Disable projects module.
3.Event PROP_COOKIE with null instance cookie will come.

Please evaluate if it is possible to fix.
Comment 9 mslama 2003-04-29 14:55:47 UTC
I revert back and close I checked on current dev build 200304290100
and it works fine. (It does not work on our winsys branch which is
based on BLD200303280100.) I will check it again when we merge.

*** This issue has been marked as a duplicate of 30244 ***
Comment 10 Marian Mirilovic 2003-07-22 16:42:29 UTC
verified, closed - it's duplicate.