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.
ConfigBeanNode does not override 'AbstractNode.getOpenedIcon()' so JSR-88 clients cannot provide sufficient icon support. Solution suggested by Peter Williams (pete.williams@sun.com) : Add the following method to org.netbeans.modules.j2ee.deployment.config.ui.ConfigBeanNode.java Index: ConfigBeanNode.java =================================================================== RCS file: /cvs/j2eeserver/src/org/netbeans/modules/j2ee/deployment/config/ui/ConfigBeanNode.java,v retrieving revision 1.1.2.5 diff -b -r1.1.2.5 ConfigBeanNode.java 71a72,79 > > public Image getOpenedIcon(int type) { > if(info != null) { > Image icon = info.getIcon(type); > if(icon != null) return icon; > } > return super.getOpenedIcon(type); > } Note: java.beans.BeanInfo does not support the concept of an "OpenedIcon". I'm open to suggestions as to how to enhance this in a suitable manner so that JSR-88 clients can provide both closed and open icons. However, the above method is required regardless, as the treeview uses it's own default icon (a white box) if an opened icon is not provided.
Checked in the fix as suggested. As for support for opened/closed types icon, if needed we can use the netbeans-deployments file mechanism as we did with config bean helpid.
Reduced to P3 since the remaining issue item is just support for different icons b/w opened/closed.
With introduction ConfigBeanProperties API specifying of the opened icon could be specified as a property like "openedIcon". Will put in support for it in future release.
Nam plans to add this into DConfigBeanProperties.
The fix is ready, but it would affect existing AS plugin. The functionality is not needed for 4.0. Will checkin the fix for 4.1.
Nam: Do we have the fix now integrated in 4.1?
I did not get a chance. This will now need API changes review. I don't think it worth the trouble?
I agree with Nam - this is not critical for 4.1.
P4 as what we have in 4.1 is ok for the AS plugin. Could improve in 4.2.
closing as WONTFIX since the j2eeserver does no longer have control over server specific data objects.