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 2011/01/04 04:40:03 UTC

svn commit: r1054883 - /cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java

Author: dkulp
Date: Tue Jan  4 03:40:03 2011
New Revision: 1054883

URL: http://svn.apache.org/viewvc?rev=1054883&view=rev
Log:
[CXF-2697] If order.size() == 0, allow the empty params.
Patch from Dieter Freismuth applied

Modified:
    cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java

Modified: cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java?rev=1054883&r1=1054882&r2=1054883&view=diff
==============================================================================
--- cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java (original)
+++ cxf/trunk/rt/core/src/main/java/org/apache/cxf/interceptor/URIMappingInterceptor.java Tue Jan  4 03:40:03 2011
@@ -134,11 +134,7 @@ public class URIMappingInterceptor exten
     protected Map<String, String> keepInOrder(Map<String, String> params, 
                                               OperationInfo operation,
                                               List<String> order) {
-        if (params == null) {
-            return params;
-        }
-        
-        if (order == null || order.size() == 0) {
+        if (params == null || order == null) {
             return params;
         }
                 
@@ -148,12 +144,12 @@ public class URIMappingInterceptor exten
         }
         
         if (order.size() != params.size()) {
-            LOG.info(order.size()
+            LOG.fine(order.size()
                      + " parameters definded in WSDL but found " 
                      + params.size() + " in request!");            
             Collection rest = CollectionUtils.diff(order, params.keySet());
             if (rest != null && rest.size() > 0) {
-                LOG.info("Set the following parameters to null: " + rest);
+                LOG.fine("Set the following parameters to null: " + rest);
                 for (Iterator iter = rest.iterator(); iter.hasNext();) {
                     String key = (String)iter.next();
                     orderedParameters.put(key, null);