Typically if StartServer is not provided we would
get NPE in various places. The correct behaviour
is to check for null whereever StartServer is used
and decide what should be default behavior. For
example if StartServer is truely optional, when it
is not provided IDE should assume the server is
running (somehow started manually) and continue
with deployment. In case this assumption turned
false, exception need to be handle gracefully.
A thourough review for other optional services
like IncrementalDeploy, FindJSPServlet and
TargetModuleIDResolver should also be done.
Nam, does this somehow manifest itself to the user currently?
This bug would not affect any current plugins. It just make
development of new plugin in initial phase harder where people trying
out their initial plugin that does not yet have an implemmentation of
Since this does not affect current users and no additional plugins are
planned at this point, can this be downgraded to P4?
Will solve in 4.1 as part of effort to simplify plugin development.
does not effect current plugins -> marking as enhancement for promoF
when we want to add support for more plugins
TM 5.0 -> TBD
this is really a TASK item...
nnguyen - please resolve or re-assign
Reassign to default. The request was too old and code has changed, not sure if this is still applicable in current context. I recommend new reviewer close as won't fix.