You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jackrabbit.apache.org by an...@apache.org on 2008/02/19 08:47:42 UTC

svn commit: r629010 - /jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java

Author: angela
Date: Mon Feb 18 23:47:34 2008
New Revision: 629010

URL: http://svn.apache.org/viewvc?rev=629010&view=rev
Log:
JCR-1392: Exception not thrown where it appears it should have been (Patch provided by Dave Brosius. Thanks!)

Modified:
    jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java

Modified: jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java?rev=629010&r1=629009&r2=629010&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-server/src/main/java/org/apache/jackrabbit/webdav/jcr/search/SearchResultProperty.java Mon Feb 18 23:47:34 2008
@@ -93,7 +93,7 @@
         } else if (propValue instanceof Element) {
             parseColumnElement((Element)property.getValue(), colList, valList, valueFactory);
         } else {
-            new IllegalArgumentException("SearchResultProperty requires a list of 'dcr:column' xml elements.");
+            throw new IllegalArgumentException("SearchResultProperty requires a list of 'dcr:column' xml elements.");
         }
 
         columnNames = (String[]) colList.toArray(new String[colList.size()]);
@@ -209,4 +209,4 @@
         }
         return elem;
     }
-}
\ No newline at end of file
+}