Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 410843

Summary: Unsupported servers cannot be removed from list of generic client runtimes
Product: [WebTools] WTP Webservices Reporter: Keith Chong <keith.chong.ca>
Component: jst.wsAssignee: Keith Chong <keith.chong.ca>
Status: RESOLVED FIXED QA Contact: Keith Chong <keith.chong.ca>
Severity: normal    
Priority: P3 CC: cbridgha, jcayne
Version: 3.2.5   
Target Milestone: 3.2.5 P   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 398897    
Bug Blocks:    
Attachments:
Description Flags
Apply to 3.2.5p org.eclipse.jst.ws.consumption.ui none

Description Keith Chong CLA 2013-06-14 13:37:48 EDT
+++ This bug was initially created as a clone of Bug #398897 +++

Certain servers do not support all client runtimes which contain support for a generic project facet type, such as jst.web, for defining a client.

Example based on http://www.eclipse.org/webtools/jst/components/ws/documents/ContributingWebServiceRT.html:

<!-- define support for Axis Java clients in Web projects -->
<extension point="org.eclipse.jst.ws.consumption.ui.clientRuntimes">
    <clientRuntime id="org.eclipse.jst.ws.axis.consumption.web"
        clientImplementationTypeId="org.eclipse.jst.ws.client.type.java"
        runtimeId="org.eclipse.jst.ws.axis.creation.axisWebServiceRT"       class="org.eclipse.jst.ws.internal.axis.creation.ui.wsrt.AxisWebServiceRuntime">
        <required-facet-version facet="jst.web" version="2.3" allow-newer="true"/>
    </clientRuntime>
</extension> 

Looking at the example, any server can be selected which supports the jst.web facet. There should be a mechanism for removing an unsupported server which uses the same runtimeId. Using the runtime checker is too late in the process as the validation completes after the wizard dialog is closed.


Attached a proposed patch for adding an optional attribute for restricting the servers supported by the extension. This attribute is optional so as to not break existing adopters.
Comment 1 Keith Chong CLA 2013-06-14 13:43:26 EDT
Created attachment 232393 [details]
Apply to 3.2.5p org.eclipse.jst.ws.consumption.ui
Comment 2 Keith Chong CLA 2013-06-14 13:44:52 EDT
The patch includes the fix for 410844 as well since it is related.  It excludes any version and feature updates.
Comment 3 Keith Chong CLA 2013-08-15 12:05:29 EDT
Delivered to 325p