You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by th...@apache.org on 2013/10/30 14:38:59 UTC

svn commit: r1537084 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java

Author: thomasm
Date: Wed Oct 30 13:38:59 2013
New Revision: 1537084

URL: http://svn.apache.org/r1537084
Log:
OAK-1106 Query engine does not deal with remapped namespaces (improved error reporting)

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java?rev=1537084&r1=1537083&r2=1537084&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryImpl.java Wed Oct 30 13:38:59 2013
@@ -631,7 +631,11 @@ public class QueryImpl implements Query 
         if (namePathMapper == null) {
             return path;
         }
-        return namePathMapper.getOakPath(path);
+        String p = namePathMapper.getOakPath(path);
+        if (p == null) {
+            throw new IllegalArgumentException("Invalid path or namespace prefix: " + path);
+        }
+        return p;
     }
 
     @Override