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 38185

Summary:	ExplorerUtils replacement for ExplorerPanel does not set activatedNodes
Product:	platform	Reporter:	Jaroslav Tulach <jtulach>
Component:	Explorer	Assignee:	Jaroslav Tulach <jtulach>
Status:	VERIFIED FIXED
Severity:	blocker	CC:	apireviews, jrechtacek, jtulach
Priority:	P3	Keywords:	API_REVIEW_FAST
Version:	3.x
Hardware:	PC
OS:	Linux
Issue Type:	DEFECT	Exception Reporter:
Attachments:	Improvements to code and test base

Description Jaroslav Tulach 2003-12-17 14:39:27 UTC

As has been discussed on dev@openide:
http://openide.netbeans.org/servlets/ReadMsg?msgId=636478&listName=dev

the default way of using ExplorerUtils leaves
TopComponent.getLookup() and
TopComponent.getActivatedNodes() in inconsistent
state. 

I'am attaching a solution that fixes this state by
adding one new TopComponent constructor with
Lookup as argument. If used, the TopComponent then
automatically synchronizes lookup content with its
getActivatedNodes values.

Comment 1 Jaroslav Tulach 2003-12-17 14:42:39 UTC

Created attachment 12614 [details]
Improvements to code and test base

Comment 2 Jaroslav Tulach 2003-12-22 16:45:40 UTC

Silence == Agreement. Will be commited soon.

Comment 3 Jaroslav Tulach 2003-12-27 18:02:22 UTC

Checking in openide-spec-vers.properties;
/cvs/openide/openide-spec-vers.properties,v  <-- 
openide-spec-vers.properties
new revision: 1.127; previous revision: 1.126
done
Processing log script arguments...
More commits to come...
Checking in api/doc/changes/apichanges.xml;
/cvs/openide/api/doc/changes/apichanges.xml,v  <--  apichanges.xml
new revision: 1.180; previous revision: 1.179
done
Processing log script arguments...
More commits to come...
Checking in src/org/openide/explorer/ExplorerUtils.java;
/cvs/openide/src/org/openide/explorer/ExplorerUtils.java,v  <-- 
ExplorerUtils.java
new revision: 1.3; previous revision: 1.2
done
Processing log script arguments...
More commits to come...
Checking in src/org/openide/windows/TopComponent.java;
/cvs/openide/src/org/openide/windows/TopComponent.java,v  <-- 
TopComponent.java
new revision: 1.109; previous revision: 1.108
done
Processing log script arguments...
More commits to come...
Checking in
test/unit/src/org/openide/explorer/ExplorerUtilCreateLookupTest.java;
/cvs/openide/test/unit/src/org/openide/explorer/ExplorerUtilCreateLookupTest.java,v
 <--  ExplorerUtilCreateLookupTest.java
new revision: 1.2; previous revision: 1.1
done
Processing log script arguments...
More commits to come...
RCS file:
/cvs/openide/test/unit/src/org/openide/windows/TopComponentLookupToNodesBridge.java,v
done
Checking in
test/unit/src/org/openide/windows/TopComponentLookupToNodesBridge.java;
/cvs/openide/test/unit/src/org/openide/windows/TopComponentLookupToNodesBridge.java,v
 <--  TopComponentLookupToNodesBridge.java
initial revision: 1.1

Comment 4 Marian Mirilovic 2005-07-13 13:25:35 UTC

closed