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 36441

Summary:	editors settings are lost
Product:	platform	Reporter:	andrew <andrew>
Component:	Nodes	Assignee:	Jaroslav Tulach <jtulach>
Status:	VERIFIED FIXED
Severity:	blocker	CC:	jtulach
Priority:	P1
Version:	3.x
Hardware:	PC
OS:	Linux
Issue Type:	DEFECT	Exception Reporter:
Attachments:	ide.log is attached, as Mato have suggested in nbdev list Yes. Here is a debug message from Node.setChildren method.

Description andrew 2003-10-06 09:02:49 UTC

Env:

Build from this morning CVS dev. trunk,
j2se 1.4.2, Linux 2.4.22.

Steps:

1. build "standard"
2. run NB with empty user dir
3. skip project importing
4. go to "Options->Editing->Editor settings" tree 
node

Result:

The node hasn't any child. It is impossible to set
options for any (java/text/xml/...) editor.

Andrew

Comment 1 andrew 2003-10-06 09:14:57 UTC

Created attachment 11799 [details]
ide.log is attached, as Mato have suggested in nbdev list

Comment 2 Martin Roskanin 2003-10-06 09:34:16 UTC

Thanks. I have thought that ide.log will provide some exception info
that caused the Editor Settings node problem. But, unfortunatelly, it
didn't. 
Tracking the problem I have found that it occurs only since the build
from 06.10. The build 03.10 seems to be OK. I think that the problem
could be introduced by the change of FilterNode in openide. (Editor
settings node is FilterNode)
http://openide.netbeans.org/source/browse/openide/src/org/openide/nodes/FilterNode.java.diff?r1=1.81&r2=1.82

Yarda, do we need some change in editor module in accordance with the
change in FilterNode? I am trying to find out what went wrong...

Comment 3 Jaroslav Tulach 2003-10-06 10:58:24 UTC

It is surprising that this FilterNode change affects you. Are you sure
that lines in the diff are really invoked? Is setChildren method
really called?

Comment 4 Martin Roskanin 2003-10-06 13:25:44 UTC

Created attachment 11802 [details]
Yes. Here is a debug message from Node.setChildren method.

Comment 5 Jaroslav Tulach 2003-10-06 17:40:14 UTC

Thanks for evaluation. It is really bug in nodes. I am taking over.

Comment 6 Jaroslav Tulach 2003-10-06 17:43:12 UTC

Will be easy fix I guess.

Comment 7 Jaroslav Tulach 2003-10-06 18:04:50 UTC

Fixed and regression test written:

/cvs/openide/src/org/openide/nodes/FilterNode.java,v  <--  FilterNode.java
new revision: 1.83; previous revision: 1.82
/cvs/openide/test/unit/src/org/openide/nodes/FilterNodeTest.java,v 
<--  FilterNodeTest.java
new revision: 1.9; previous revision: 1.8

Comment 8 andrew 2003-10-06 18:42:43 UTC

Verified.

Thanks! Indeed very quick fix :-)

Andrew

Comment 9 Jaroslav Tulach 2003-10-07 08:28:16 UTC

Simple mistake + quick bugreport = quick fix.