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.
Certain servers require deployment descriptor even if it is not required by the specification. There should be a way for plugin to report that.
I already pushed the change to web-main 3dd074f7fca4. I'll adapt the change in case there will be any objections. http://hg.netbeans.org/web-main/rev/3dd074f7fca4
Please review.
DK01 - you turned add/remove listener methods in spi/J2eeModuleProvider.java public but patch does not uses them anyway. Was that intentional? DK02 - this new API will be used only during project creation right? It will not catch cases when project is created for GF and deployed to Weblogic. Switching server is corner case scenario though so perhaps it is not an issue.
(In reply to comment #3) > DK01 - you turned add/remove listener methods in spi/J2eeModuleProvider.java > public but patch does not uses them anyway. Was that intentional? It is intentional, but should not be part of the patch. Note that there is no real change as these methods are declared in interface. The change was made just to make them consistent with other methods in the interface which are explicitly declared public. > > DK02 - this new API will be used only during project creation right? It will > not catch cases when project is created for GF and deployed to Weblogic. It will. It handles the server change. Note that the whole machinery works for EAR only as it is now. I'm not sure whether it is worth to implement for other project types unless it is really needed (should be easy for web, not sure about ejb and appclient). > Switching server is corner case scenario though so perhaps it is not an issue. I agree.