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 175202 - API introduced in issue 151716 doesn't work correctly
Summary: API introduced in issue 151716 doesn't work correctly
Status: RESOLVED FIXED
Alias: None
Product: javame
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 6.x
Hardware: Macintosh Mac OS X
: P2 blocker (vote)
Assignee: _ tboudreau
URL:
Keywords:
Depends on:
Blocks: 162137
  Show dependency tree
 
Reported: 2009-10-22 13:04 UTC by martin_sladecek
Modified: 2009-11-13 12:18 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Sample code (3.35 KB, text/plain)
2009-10-22 13:18 UTC, martin_sladecek 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description martin_sladecek 2009-10-22 13:04:34 UTC 
API from issue 151716 allows addition of subnodes to mobility project, but firing a Change Event doesn't work. The change 
(adding or removing a node) isn't visible.
Comment 1 martin_sladecek 2009-10-22 13:18:28 UTC 
Created attachment 89916 [details]
Sample code
Comment 2 Petr Suchomel 2009-11-06 09:57:25 UTC 
Please make this work with Tim next week when he's in or office
Comment 3 _ tboudreau 2009-11-12 04:04:10 UTC 
FYI, I now have a failing test for this problem - looks like the problem actually may be somewhere in Nodes.  Should have a fix soon.
Comment 4 _ tboudreau 2009-11-12 06:09:38 UTC 
The problem is:  We are actually using a single key, ChildKind.Foreign, in our Children object, for all nodes from other modules.  When a change event is fired, we refresh the keys.  But of course, the key list has not really changed - so the underlying Children reuses its existing Nodes, instead of creating new ones.

Should be simple enough to fix, just have to decide how.
Comment 5 _ tboudreau 2009-11-12 06:43:42 UTC 
Fixed in main/ changeset 70a0130084fb, regression tests added
Comment 6 _ tboudreau 2009-11-12 07:12:52 UTC 
Fix polished further in 951fcbc170fb to use keys from attached NodeLists directly, to avoid briefly clearing the keys of the project node
Comment 7 _ tboudreau 2009-11-13 12:18:29 UTC 
x