You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2008/05/12 21:24:59 UTC
svn commit: r655612 - in /cxf/branches/2.0.x-fixes: ./
api/src/main/java/org/apache/cxf/service/model/BindingInfo.java
rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java
Author: dkulp
Date: Mon May 12 12:24:59 2008
New Revision: 655612
URL: http://svn.apache.org/viewvc?rev=655612&view=rev
Log:
Merged revisions 655602 via svnmerge from
https://svn.apache.org/repos/asf/cxf/trunk
........
r655602 | dkulp | 2008-05-12 15:06:32 -0400 (Mon, 12 May 2008) | 2 lines
Fix more issues with wsdl first problems where binding operations have message names, but the portTypes don't (and thus should use defaults)
........
Modified:
cxf/branches/2.0.x-fixes/ (props changed)
cxf/branches/2.0.x-fixes/api/src/main/java/org/apache/cxf/service/model/BindingInfo.java
cxf/branches/2.0.x-fixes/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java
Propchange: cxf/branches/2.0.x-fixes/
------------------------------------------------------------------------------
Binary property 'svnmerge-integrated' - no diff available.
Modified: cxf/branches/2.0.x-fixes/api/src/main/java/org/apache/cxf/service/model/BindingInfo.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.0.x-fixes/api/src/main/java/org/apache/cxf/service/model/BindingInfo.java?rev=655612&r1=655611&r2=655612&view=diff
==============================================================================
--- cxf/branches/2.0.x-fixes/api/src/main/java/org/apache/cxf/service/model/BindingInfo.java (original)
+++ cxf/branches/2.0.x-fixes/api/src/main/java/org/apache/cxf/service/model/BindingInfo.java Mon May 12 12:24:59 2008
@@ -64,19 +64,22 @@
return name;
}
- private boolean nameEquals(String a, String b) {
+ private boolean nameEquals(String a, String b, String def) {
if (a == null) {
// in case of input/output itself is empty
return true;
} else {
+ if (b == null) {
+ b = def;
+ }
return "".equals(a) ? "".equals(b) : a.equals(b);
}
}
public BindingOperationInfo buildOperation(QName opName, String inName, String outName) {
for (OperationInfo op : getInterface().getOperations()) {
if (opName.equals(op.getName())
- && nameEquals(inName, op.getInputName())
- && nameEquals(outName, op.getOutputName())) {
+ && nameEquals(inName, op.getInputName(), op.getName().getLocalPart() + "Request")
+ && nameEquals(outName, op.getOutputName(), op.getName().getLocalPart() + "Response")) {
return new BindingOperationInfo(this, op);
}
Modified: cxf/branches/2.0.x-fixes/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java
URL: http://svn.apache.org/viewvc/cxf/branches/2.0.x-fixes/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java?rev=655612&r1=655611&r2=655612&view=diff
==============================================================================
--- cxf/branches/2.0.x-fixes/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java (original)
+++ cxf/branches/2.0.x-fixes/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/SimpleMethodDispatcher.java Mon May 12 12:24:59 2008
@@ -19,7 +19,6 @@
package org.apache.cxf.frontend;
import java.lang.reflect.Method;
-import java.util.HashMap;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
@@ -46,7 +45,8 @@
for (Method m : methods) {
methodToOp.put(m, o);
- Map<BindingInfo, BindingOperationInfo> biToBop = new HashMap<BindingInfo, BindingOperationInfo>();
+ Map<BindingInfo, BindingOperationInfo> biToBop
+ = new ConcurrentHashMap<BindingInfo, BindingOperationInfo>();
infoMap.put(m, biToBop);
}