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: | Make OptionalDeploymentManagerFactory.getAddInstanceIterator() abstract | ||
|---|---|---|---|
| Product: | serverplugins | Reporter: | jlaskowski <jlaskowski> |
| Component: | Code | Assignee: | issues@serverplugins <issues> |
| Status: | REOPENED --- | ||
| Severity: | normal | ||
| Priority: | P3 | ||
| Version: | 6.x | ||
| Hardware: | PC | ||
| OS: | Mac OS X | ||
| Issue Type: | ENHANCEMENT | Exception Reporter: | |
|
Description
jlaskowski
2010-05-09 21:29:34 UTC
That would be incompatible API change. I don't mean to play "do this vs I won't because I said so" game, but would you consider a change that would eventually helped pinpoint such an issue that a user is stuck with debugging trying to figure out what methods are responsible for a given task, i.e. the aforementioned method for showing a new server entry in the Add Server panel. WDYT? Close it if you think it's not doable. It is enhancement. Feel free to attach patch that will preserve API compatibility. What about throwing a RuntimeException as a default implementation with a meaningful message why it's important to override it? You still suppose that it is mandatory to implement this method. The plugin is not required to provide the wizard (such as different embedded runtimes etc.). You would immediately break such plugins with your change. |