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 146786 - Stop using deprecated EnhancedCustomPropertyEditor
Summary: Stop using deprecated EnhancedCustomPropertyEditor
Status: NEW
Alias: None
Product: serverplugins
Classification: Unclassified
Component: Sun Appserver 9 (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: Vince Kraemer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-09-10 14:25 UTC by Nitya Doraisamy
Modified: 2017-02-16 16:17 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
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 Nitya Doraisamy 2008-09-10 14:26:00 UTC 
EnhancedCustomPropertyEditor is being used in sun/appsrv/src/org/netbeans/modules/j2ee/sun/ide/editors/ui/DDTablePanel.java

It has been deprecated and needs to be removed.
Comment 1 Nitya Doraisamy 2008-09-10 14:27:44 UTC 
See Issue 75193 for more info.

The NameValue editor in the plugin should be redone and the code using EnhancedCustomPropertyEditor is part of this editor.
Comment 2 Nitya Doraisamy 2008-12-09 05:56:09 UTC 
The property editor rewrite is planned for 7.0 but is targeted for M2 or later.
Comment 3 Vince Kraemer 2009-09-16 23:55:28 UTC 
Assigned engineer: please correct the tm value.  nb 6.7 has shipped, so a tm of
6.7  is invalid at this point
Comment 4 Nitya Doraisamy 2009-10-26 18:43:45 UTC 
Will not happen for 6.8 release. resetting target to future
Comment 5 bruehlicke 2017-02-16 16:17:09 UTC 
Not obvious how to replace the usage of EnhancedCustomPropertyEditor with PropertyEnv. Not critical at all (as long as we keep the code), but could be exciting to see how to get this out of the DDTablePanel. Removing it for sure stops the call to setting the value of the property entered in a custom Editor based on the DDTablePanel. [Still there in NB 8.2, Jan 2017]