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

Bug 293685

Summary: JAX-RPC client JSPs: JSPG0036E: Failed to find resource /sampleEchoProxy/TestClient.jsp
Product: [WebTools] WTP Java EE Tools Reporter: Neeraj Agrawal <nagrawal>
Component: jst.j2eeAssignee: jst.j2ee <jst.j2ee-inbox>
Status: RESOLVED FIXED QA Contact: Chuck Bridgham <cbridgha>
Severity: normal    
Priority: P3 CC: ccc, jsholl, yenlu
Version: 3.0.5Flags: cbridgha: review+
Target Milestone: 3.0.5 P   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch none

Description Neeraj Agrawal CLA 2009-10-29 12:10:54 EDT 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3
Build Identifier: 

When Web servicee client wizard is used to generate a jax-rpc client and sample JSPs, this results in a http 404 error: JSPG0036E: Failed to find resource /sampleEchoProxy/TestClient.jsp. The reason is the output folder for a web project is not  <root folder>/WEB-INF/classes".

Reproducible: Always
Comment 1 Neeraj Agrawal CLA 2009-10-29 12:16:09 EDT 
Created attachment 150839 [details]
Patch

The reason of problem is 291225 which removed the code from WebFacetInstallDelegate to fix the output folder as desired. 

The Web service code is using the mechanism to create only WebFacetInstallDataModelProvider and doesn't create and sets the properties on JavaFacetInstallDataModelProvider.  Both of those providers are required to make project creation work properly. But, since this is behavior change, i am putting the code back in WebFacetInstallDelegate  to fix the Web project's output folder.
For 32, we would let web services to properly instantiate correct objects.
Comment 2 Chuck Bridgham CLA 2009-10-30 09:35:30 EDT 
approve
Comment 3 Carl Anderson CLA 2009-10-30 21:39:10 EDT 
Committed to R3_0_5_patches
Comment 4 Jason Sholl CLA 2009-11-10 16:40:36 EST 
Checked into 31M.
Please provide a patch for 32; this one doesn't apply properly.
Comment 5 Jason Sholl CLA 2009-11-10 16:43:38 EST 
never mind about the 32 patch; didn't see the previous comment.