Thanks Bilgin,

Will take a look in a few hours:

I think the following patch may be needed also:

### Eclipse Workspace Patch 1.0
#P ofbiz
Index: framework/webapp/src/org/ofbiz/webapp/event/SOAPEventHandler.java
===================================================================
--- framework/webapp/src/org/ofbiz/webapp/event/SOAPEventHandler.java (revision 894372) +++ framework/webapp/src/org/ofbiz/webapp/event/SOAPEventHandler.java (working copy)
@@ -227,6 +227,7 @@
private void sendError(HttpServletResponse res, String errorMessage) throws EventHandlerException {
        try {
            // setup the response
+            res.setContentType("text/xml");
            Map<String, Object> results = FastMap.newInstance();
            results.put("errorMessage", errorMessage);
            String xmlResults= XmlSerializer.serialize(results);

Cheers,

Chris

Bilgin Ibryam wrote:
Christopher Snow wrote:
Hi Hans,

I've been looking through the code and it looks like a nice job. I am struggling with a couple of issues though:

1) The generated wsdl causes tools (e.g. wsdl2java) to put a namespace prefix on the children of map-Map. Unfortunately, the SOAPEventHandler and XmlSerializer.derserialize method don't appear to be able to handle the elements with a namespace prefix.

Chris, I have just committed a fix for this issue in r894359 to ofbiz trunk. Now the namespace are handled fine.


2) Because the passed in parameters are not recognized, I am receiving service errors. However, the response appears to have multiple error messages and not follow the <ServiceName>Response pattern - I.e. the error response is just <Response>

I didn't notice that. I will take a look soon.
Bilgin
I have documented these issues in: OFBIZ-3385 - I'm currently looking though the code to try and find a resolution, although I think the first issue may need a change in the way the wsdl is generated.

Many thanks,

Chris



--
Chris Snow - CEng MBCS CITP MBA (Tech Mgmt) (Open) CISSP

Tel: 01453 890660
Mob: 07944 880950
Www: www.snowconsulting.co.uk

Reply via email to