You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by mb...@apache.org on 2008/06/23 22:12:44 UTC

svn commit: r670728 - /commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java

Author: mbenson
Date: Mon Jun 23 13:12:44 2008
New Revision: 670728

URL: http://svn.apache.org/viewvc?rev=670728&view=rev
Log:
potential NPE

Modified:
    commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java

Modified: commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java?rev=670728&r1=670727&r2=670728&view=diff
==============================================================================
--- commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java (original)
+++ commons/proper/jxpath/trunk/src/java/org/apache/commons/jxpath/util/ValueUtils.java Mon Jun 23 13:12:44 2008
@@ -50,10 +50,10 @@
      * @return boolean
      */
     public static boolean isCollection(Object value) {
+        value = getValue(value);
         if (value == null) {
             return false;
         }
-        value = getValue(value);
         if (value.getClass().isArray()) {
             return true;
         }