You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jo...@apache.org on 2007/09/15 04:07:04 UTC

svn commit: r575853 - /ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java

Author: jonesde
Date: Fri Sep 14 19:07:03 2007
New Revision: 575853

URL: http://svn.apache.org/viewvc?rev=575853&view=rev
Log:
Small validation improvement

Modified:
    ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java

Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java?rev=575853&r1=575852&r2=575853&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java (original)
+++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/UtilXml.java Fri Sep 14 19:07:03 2007
@@ -421,6 +421,7 @@
      * returns the first element. */
     public static Element firstChildElement(Element element, String childElementName) {
         if (element == null) return null;
+        if (UtilValidate.isEmpty(childElementName)) return null;
         // get the first element with the given name
         Node node = element.getFirstChild();
 
@@ -429,7 +430,6 @@
                 if (node.getNodeType() == Node.ELEMENT_NODE && (childElementName == null ||
                         childElementName.equals(node.getNodeName()))) {
                     Element childElement = (Element) node;
-
                     return childElement;
                 }
             } while ((node = node.getNextSibling()) != null);