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.
| Summary: | Allow change of separator for StringArrayEditor | ||
|---|---|---|---|
| Product: | platform | Reporter: | Jaroslav Tulach <jtulach> |
| Component: | Property Editors | Assignee: | Jaroslav Tulach <jtulach> |
| Status: | RESOLVED FIXED | ||
| Severity: | blocker | CC: | apireviews, ludo |
| Priority: | P3 | Keywords: | API_REVIEW_FAST |
| Version: | 4.x | ||
| Hardware: | All | ||
| OS: | All | ||
| Issue Type: | ENHANCEMENT | Exception Reporter: | |
| Attachments: | Test, fix and api changes | ||
Created attachment 20778 [details]
Test, fix and api changes
I plan to integrate on Feb18, but if reviewers agree, I'd be happy to integrate sooner. Feb18? :-) I would really prefer to integrate before March18, if this is what you mean... I tested it on my laptop, and so far, I am fine with the change and the test. I am sorry for my "date" confusion. As there were no other comments, I will integrate tomorrow. cvs ci -m "#56257: StringArrayEditor honors item.separator" Checking in openide/openide-spec-vers.properties; /cvs/openide/openide-spec-vers.properties,v <-- openide-spec-vers.properties new revision: 1.168; previous revision: 1.167 done Checking in openide/api/doc/changes/apichanges.xml; /cvs/openide/api/doc/changes/apichanges.xml,v <-- apichanges.xml new revision: 1.236; previous revision: 1.235 done Checking in openide/api/doc/org/openide/explorer/doc-files/api.html; /cvs/openide/api/doc/org/openide/explorer/doc-files/api.html,v <-- api.html new revision: 1.51; previous revision: 1.50 done Checking in openide/arch/arch-openide-propertysheet.xml; /cvs/openide/arch/arch-openide-propertysheet.xml,v <-- arch-openide-propertysheet.xml new revision: 1.23; previous revision: 1.22 done Checking in core/src/org/netbeans/beaninfo/editors/StringArrayEditor.java; /cvs/core/src/org/netbeans/beaninfo/editors/StringArrayEditor.java,v <-- StringArrayEditor.java new revision: 1.7; previous revision: 1.6 done RCS file: /cvs/core/test/unit/src/org/netbeans/beaninfo/editors/StringArrayEditorTest.java,v done Checking in core/test/unit/src/org/netbeans/beaninfo/editors/StringArrayEditorTest.java; /cvs/core/test/unit/src/org/netbeans/beaninfo/ |
Ludo's team is using StringArrayEditor, but they would like to change the default separator (",") for something else. This can be easily suplied by ExPropertyEditor's properties mechanism. I'll attach diff with proposed changes.