Bug 221854 - Merging and saving of webservices.xml removes <wsdl-port> element
Summary: Merging and saving of webservices.xml removes <wsdl-port> element
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0.3   Edit
Assignee: Neeraj Agrawal CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-03-07 09:33 EST by Neeraj Agrawal CLA
Modified: 2008-09-30 01:57 EDT (History)
1 user (show)

See Also:
cbridgha: review+

Attachments
Patch (2.35 KB, text/plain)
2008-03-07 09:38 EST, Neeraj Agrawal CLA 		no flags Details
Proposed patch (895 bytes, patch)
2008-09-25 13:31 EDT, Neeraj Agrawal CLA 		bjorn.freeman-benson: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Neeraj Agrawal CLA 2008-03-07 09:33:09 EST 
Build ID: 3.4 M5

Steps To Reproduce:
A regression in M5 has occured when users attempt to regenerate or update their JAX-RPC Web services through the Web Service wizard. 

The Web Service wizard code calls the Web Services JAX-RPC emitters to generate a file outside the workspace. 
This file is later read to populate a temporary model and its content are mergeg into the webservices.xml that is in the user's workspace. The updated webservices.xml contains a <webservice-description> whose <port-component>'s <wsdl-port> element is missing. 

More information:
Comment 1 Neeraj Agrawal CLA 2008-03-07 09:38:47 EST 
Created attachment 91872 [details]
Patch

The patch fixes the problem by properly reading the schema version.hh
Comment 2 Chuck Bridgham CLA 2008-03-10 15:56:55 EDT 
Dropped last weeks I build
Comment 3 Carl Anderson CLA 2008-03-11 12:01:58 EDT 
Didn't notice Chuck's comment, committed this patch to HEAD- Chuck - let me know if this is a problem, and I will revert the change.
Comment 4 Neeraj Agrawal CLA 2008-04-03 15:38:02 EDT 
verified and closing
Comment 5 Neeraj Agrawal CLA 2008-09-25 11:00:36 EDT 
This is still a problem when the workspace has "space" in it.
Comment 6 Neeraj Agrawal CLA 2008-09-25 13:31:53 EDT 
Created attachment 113495 [details]
Proposed patch
Comment 7 Chuck Bridgham CLA 2008-09-26 10:10:07 EDT 
simple fix to preserver space....   approved
Comment 8 Carl Anderson CLA 2008-09-30 01:57:13 EDT 
Committed to R3_0_maintenance for 3.0.3