Return-Path: Delivered-To: apmail-incubator-cxf-issues-archive@locus.apache.org Received: (qmail 93587 invoked from network); 9 Nov 2006 03:27:06 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 9 Nov 2006 03:27:06 -0000 Received: (qmail 64737 invoked by uid 500); 9 Nov 2006 03:27:17 -0000 Delivered-To: apmail-incubator-cxf-issues-archive@incubator.apache.org Received: (qmail 64552 invoked by uid 500); 9 Nov 2006 03:27:16 -0000 Mailing-List: contact cxf-issues-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cxf-dev@incubator.apache.org Delivered-To: mailing list cxf-issues@incubator.apache.org Received: (qmail 64533 invoked by uid 99); 9 Nov 2006 03:27:16 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 08 Nov 2006 19:27:16 -0800 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: domain of James.Mao@iona.com designates 65.223.216.181 as permitted sender) Received: from [65.223.216.181] (HELO amereast-smg1.iona.com) (65.223.216.181) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 08 Nov 2006 19:27:03 -0800 Received: from amer-ems1.IONAGLOBAL.COM ([10.65.6.25]) by amereast-smg1.iona.com (Switch-3.1.7/Switch-3.1.7) with ESMTP id kA93QFp1016920; Wed, 8 Nov 2006 22:26:15 -0500 (EST) Received: from [10.129.9.116] ([10.129.9.116]) by amer-ems1.IONAGLOBAL.COM with Microsoft SMTPSVC(6.0.3790.1830); Wed, 8 Nov 2006 22:26:41 -0500 Message-ID: <45529F9C.20207@iona.com> Date: Thu, 09 Nov 2006 11:25:16 +0800 From: James Mao User-Agent: Thunderbird 1.5.0.7 (Windows/20060909) MIME-Version: 1.0 To: cxf-dev@incubator.apache.org CC: cxf-issues@incubator.apache.org Subject: Re: [jira] Commented: (CXF-161) wsdl2java doesn't map parameter order correctly References: <633182.1162930732052.JavaMail.jira@brutus> In-Reply-To: <633182.1162930732052.JavaMail.jira@brutus> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-OriginalArrivalTime: 09 Nov 2006 03:26:41.0682 (UTC) FILETIME=[DFC5BF20:01C703AE] X-Virus-Checked: Checked by ClamAV on apache.org Hi Peter, Just commit the fix, can you try it again. Thanks, James. Peter Jones (JIRA) 写道: > [ http://issues.apache.org/jira/browse/CXF-161?page=comments#action_12447917 ] > > Peter Jones commented on CXF-161: > --------------------------------- > > This is still a bug. Although the header2.wsdl that's in svn passes the test that was added for cxf-161, that wsdl differs from the one attached to this issue because the header_info part is missing from the response message in the copy in svn. If I copy the file attached to this issue into tools/wsdl2java/src/test/resources/wsdl2java_wsdl/bug161/header2.wsdl, then the test that was added fails... because the parameter order is wrong. > > >> wsdl2java doesn't map parameter order correctly >> ----------------------------------------------- >> >> Key: CXF-161 >> URL: http://issues.apache.org/jira/browse/CXF-161 >> Project: CXF >> Issue Type: Bug >> Components: Tooling >> Affects Versions: 2.0-M1 >> Reporter: Peter Jones >> Assigned To: jimma >> Fix For: 2.0-M1 >> >> Attachments: header2.wsdl >> >> >> According to the "Parameter Order" section of jaxws spec, unlisted parameters mapped from in and in/out parts appear in the same order the corresponding parts appear in the input message. >> I'll attach a test wsdl in which wsdl2java doesn't follow this rule. Basically, the wsdl input message is: >> ... >> >> >> >> >> (header_info first, the_request second) - but the generated method signature >> has theRequest parameter first and headerInfo second: >> ... >> public org.apache.header2.HeaderResponse headerMethod( >> @WebParam(targetNamespace = "http://apache.org/header2", partName = "the_request", name = "Header") >> org.apache.header2.Header theRequest, >> @WebParam(targetNamespace = "http://apache.org/header2", header = true, mode = Mode.INOUT, partName = "header_info", name = "HeaderInfo") >> javax.xml.ws.Holder headerInfo >> ); >> >> > >