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.
Originally j2eeserver plugin SPI has this method: o.n.m.j2ee.deployment.plugins.api.DeploymentPlanSplitter.getDeploymentPlanFileNames(). Later on, we need a data loader for ConfigDataObject to allow both graphical and xml editing of the configuration files. The loader are registered and active very early at IDE boot and there are changes that DSP.getDeploymentPlanFileNames service is not ready yet. This entail a duplicate declaration in plugin module layer.xml. Duplication is bad in itself, but now it is worse because the DSP implementation could be part of the J2EE Platform which might not be set at all, yet the service is invoke through out. Consolidating the service into the single declaration in layer.xml not only solve this problem but also avoid confusion for new plugin writer.
If people with bigger insight to the problem agree, then it is probably good change. I'd just like to remind that the j2eeserver review http://openide.netbeans.org/tutorial/reviews/opinions_40771.html requested test coverage of XML apis: http://www.netbeans.org/issues/show_bug.cgi?id=42721 Please keep that direction and make sure your changes are covered by tests and documented in arch*xml using <api group="layer" .../>
Created attachment 19372 [details] apichanges.xml diff
Created attachment 19376 [details] checkin log for recommended unit test
Created attachment 19377 [details] checkin of apichanges, arch docs and deprecation mark in DeploymentPlanSplitter
Completed in all recommended actions.
Please update the Target Milestone whenever closing any issue.
v