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