Issue Details (XML | Word | Printable)

Key: XFIRE-417
Type: Improvement Improvement
Status: Open Open
Priority: Major Major
Assignee: Dan Diephouse
Reporter: justin
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
XFire

Load customized wsdl from classpath

Created: 24/May/06 01:38 AM   Updated: 20/Jun/06 05:50 PM
Component/s: Core
Affects Version/s: 1.1
Fix Version/s: None

Time Tracking:
Original Estimate: 5 minutes
Original Estimate - 5 minutes
Remaining Estimate: 5 minutes
Remaining Estimate - 5 minutes
Time Spent: Not Specified
Remaining Estimate - 5 minutes

File Attachments: 1. Text File ServiceBean.patch.txt (1 kB)
2. Text File ServiceBean.patch.txt (1 kB)

Environment: not platform specific


 Description  « Hide
XFire 1.1 support customized wsdl file via org.codehaus.xfire.spring.ServiceBean's wsdlURL property. However, it don't support load wsdl file from classpath. So you have to place the wsdl under some place and set the abosulate URL to ServiceBean's wsdlURL property.
XFire should extend to support load wsdl file from classpath like this:

<property name="wsdlURL" value="classpath:/cn/cetelem/cnxnet/docserver/ws/docsrv.wsdl" />

here the classpath: prefix is used to distinguish with other resource loading mechanism. This is quite spring-style and also easy to implement. See attached patch for reference. This patch only work with spring-style configure, not xbean-style. Wish the XFire developer will include this feature in the next release.



 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
justin added a comment - 24/May/06 01:48 AM
Please use this patch. There are some text formatting problems in the old one

Mike McAngus added a comment - 20/Jun/06 05:50 PM
Can't this be accomplished with @WebService(wsdlLocation=file://somepath/SomeFile.wsdl ?

Isn't "somepath" relative to the classpath?

I don't know for sure because wsdlLocation seems to be ignored no matter where I put the wsdl file, but that is what I thought should be the case.