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: | Improve ConfigurationSupport for CMP mapping | ||
---|---|---|---|
Product: | serverplugins | Reporter: | Nam Nguyen <nnguyen> |
Component: | Infrastructure | Assignee: | apireviews <apireviews> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | Keywords: | API_REVIEW_FAST |
Priority: | P1 | ||
Version: | 4.x | ||
Hardware: | PC | ||
OS: | Linux | ||
Issue Type: | ENHANCEMENT | Exception Reporter: | |
Attachments: | apichanges description |
Description
Nam Nguyen
2004-12-22 23:46:38 UTC
Since the general expectation from a wizard is user should not need to do an explicit save to persist the changes resulted from using the wizard, we also need an additional changes to give wizard code ability to commit changes to configuration DD. Here is the additional change: o.n.m.j2ee.deployment.devmodules.spi.J2eeModuleProvider.ConfigSupport { void saveConfiguration(); } Plugin could save configuration itself, but in case a wizard craete a set of new beans, multiple save is bad performance and also causing weird UI effect (e.g. flickering save all button...). Created attachment 19400 [details]
apichanges description
I would like to waive unit test of this api change due to the routing nature of the involved API's/SPI's. I have a question about signature test. Do I have take care of changing the golden file? If so, please point to location and procedure to run the test. Thanks. I would like to fast-track this api change due to 4.1 EA2 code-freeze coming up. I thought that this was always fastrack and I know the issue has been here for more than a week. From my point of view, you can go on... Described changes checked in. |