svn commit: r1201718 - /tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java

[sl...@apache.org]

Author: slaws
Date: Mon Nov 14 14:22:19 2011
New Revision: 1201718

URL: http://svn.apache.org/viewvc?rev=1201718&view=rev
Log:
Add extra check for zero child elements when processing the OMElement wrapper. 

Modified:
    tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java

Modified: tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java
URL: http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java?rev=1201718&r1=1201717&r2=1201718&view=diff
==============================================================================
--- tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java (original)
+++ tuscany/sca-java-2.x/trunk/modules/databinding-axiom/src/main/java/org/apache/tuscany/sca/databinding/axiom/OMElementWrapperHandler.java Mon Nov 14 14:22:19 2011
@@ -231,6 +231,14 @@ public class OMElementWrapperHandler imp
     private List<Object> getInvalidChildren(List<List<OMElement>> groupedElementList, List<ElementInfo> childElements) {
         List<Object> retVal = new ArrayList<Object>();
         
+        // if this is an operation without arguments then it seems that
+        // the groupedElementList can have a list with an empty list in it while
+        // the child elements list is empty. This leads to an IndexOutOfBounds when 
+        // trying to de-reference childElements. So check for this now
+        if (childElements.size() == 0){
+            return retVal;
+        }
+        
         // Since not all the ElementInfo(s) will be represented, (if some elements don't appear as children
         // of the wrapper payload, we need to loop through the schema 
         for (int index=0; index < groupedElementList.size(); index++) {