Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cxf-dev@incubator.apache.org
Received-SPF: pass (herse.apache.org: domain of James.Mao@iona.com designates
 65.223.216.181 as permitted sender)
Message-ID: <4546E35A.4040803@iona.com>
Date: Tue, 31 Oct 2006 13:47:06 +0800
From: James Mao <james.mao@iona.com>
User-Agent: Thunderbird 1.5.0.7 (Windows/20060909)
MIME-Version: 1.0
To: cxf-dev@incubator.apache.org
CC: cxf-commits@incubator.apache.org
Subject: Re: svn commit: r469376 - in /incubator/cxf/trunk/rt:
 core/src/main/java/org/apache/cxf/interceptor/
 frontend/jaxws/src/test/java/org/apache/cxf/jaxws/
References: <20061031053502.07AC91A9846@eris.apache.org>
 <4546E259.1040309@envoisolutions.com>
In-Reply-To: <4546E259.1040309@envoisolutions.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

Hi Dan,
Yes, i awared, that's why i just use it to log, i'm not failed the 
processing.
If i can not use jaxws stuff in that module , i guess we should remove 
the dependencies from the POM
Yes, i will use the messagePartInfo.getConcreteName().getLocalPart(), 
that's next story.

It's not a part of HTTP binding, and i don't think in GET we need to 
convert to the document, it'll slow down the processing.

Cheers,
James.

Dan Diephouse 写道:
> Hi James,
> Are you aware this information is in the service model? Are you also 
> aware that this information often times is NOT in the annotations - 
> for instance when someone does code first services? There should 
> hardly ever be any reason to use the jax-ws annotations outside the 
> JaxWsServiceFactory/JaxWsServiceConfiguration classes. Just do 
> key.equals(messagePartInfo.getConcreteName().getLocalPart()) to see if 
> they match.
>
> Also you might want to look at using the utility code in the HTTP 
> binding to turn parameters and their values into a document which 
> conforms to the schema.
>
> - Dan
>
>
> mmao@apache.org wrote:
>> +    private boolean isValidParameter(Annotation[][] 
>> parameterAnnotation, int index, String parameterName) {
>> +        if (parameterAnnotation == null || 
>> parameterAnnotation.length < index) {
>> +            return true;
>> +        }
>> +        Annotation[] annotations = parameterAnnotation[index];
>> +        if (annotations == null || annotations.length < 1) {
>> +            return true;
>> +        }
>> +        WebParam webParam = null;
>> +        for (Annotation annotation : annotations) {
>> +            if (annotation.annotationType() == WebParam.class) {
>> +                webParam = (WebParam) annotation;
>> +            }
>> +        }
>> +        if (webParam == null +            || 
>> StringUtils.isEmpty(webParam.name()) +            || 
>> webParam.name().equals(parameterName)) {
>> +            return true;
>> +        }
>> +        LOG.warning("The parameter name [" + parameterName 
>> +                    + "] is not match the one defined in the 
>> WebParam name [" + webParam.name() + "]");
>> +        return false;
>> +    }
>>   
> Dan Diephouse
> Envoi Solutions
> http://envoisolutions.com
> http://netzooid.com/blog
>
>