[bugfix] Handle QNames correctly in attribute names within conditions of the Range Index config

Closes eXist-db/exist#5189

Author: adamretter

extensions/indexes/range/src/main/java/org/exist/indexing/range/RangeIndexConfigAttributeCondition.java

@@ -46,6 +46,8 @@
 package org.exist.indexing.range;
 
 import org.exist.dom.QName;
+import org.exist.dom.persistent.ElementImpl;
+import org.exist.dom.persistent.NodeImpl;
 import org.exist.storage.ElementValue;
 import org.exist.storage.NodePath;
 import org.exist.util.DatabaseConfigurationException;
@@ -78,8 +80,7 @@
  */
 public class RangeIndexConfigAttributeCondition extends RangeIndexConfigCondition{
 
-    private final String attributeName;
-    private final QName attribute;
+    private final QName attributeName;
     private @Nullable final String value;
     private final Operator operator;
     private final boolean caseSensitive;
@@ -96,13 +97,23 @@ public RangeIndexConfigAttributeCondition(final Element elem, final NodePath par
                     "Range index module: Attribute condition cannot be defined for an attribute:" + parentPath);
         }
 
-        this.attributeName = elem.getAttribute("attribute");
-        if (this.attributeName.isEmpty()) {
+        final String attributeValue = elem.getAttribute("attribute");
+        if (attributeValue.isEmpty()) {
             throw new DatabaseConfigurationException("Range index module: Empty or no attribute qname in condition");
         }
 
         try {
-            this.attribute = new QName(QName.extractLocalName(this.attributeName), XMLConstants.NULL_NS_URI, QName.extractPrefix(this.attributeName), ElementValue.ATTRIBUTE);
+            final String attrLocalName = QName.extractLocalName(attributeValue);
+            @Nullable final String attrPrefix = QName.extractPrefix(attributeValue);
+            if (attrPrefix != null) {
+                @Nullable final String attrNamespace = findNamespaceForPrefix(attrPrefix, (ElementImpl) elem);
+                if (attrNamespace == null) {
+                    throw new DatabaseConfigurationException("Range index module: Missing namespace declaration for attribute qname in condition");
+                }
+                this.attributeName = new QName(attrLocalName, attrNamespace, attrPrefix, ElementValue.ATTRIBUTE);
+            } else {
+                this.attributeName = new QName(attrLocalName, XMLConstants.NULL_NS_URI, null, ElementValue.ATTRIBUTE);
+            }
         } catch (final QName.IllegalQNameException e) {
             throw new DatabaseConfigurationException("Rand index module error: " + e.getMessage(), e);
         }
@@ -171,6 +182,24 @@ public RangeIndexConfigAttributeCondition(final Element elem, final NodePath par
 
     }
 
+    private static @Nullable String findNamespaceForPrefix(final String prefix, ElementImpl contextElem) {
+        while (contextElem != null) {
+            final String namespace = contextElem.getNamespaceForPrefix(prefix);
+            if (namespace != null) {
+                return namespace;
+            }
+
+            @Nullable final Node parentNode = contextElem.getParentNode();
+            if (parentNode != null && parentNode instanceof ElementImpl) {
+                contextElem = (ElementImpl) parentNode;
+            } else {
+                contextElem = null;
+            }
+        }
+
+        return null;
+    }
+
     // lazily evaluate lowercase value to convert once when needed
     private String getLowercaseValue() {
         if (this.lowercaseValue == null) {
@@ -184,8 +213,16 @@ private String getLowercaseValue() {
 
     @Override
     public boolean matches(final Node node) {
+        final String attrValue;
+        if (attributeName.hasNamespace()) {
+            attrValue = ((Element) node).getAttributeNS(attributeName.getNamespaceURI(), attributeName.getLocalPart());
+        } else {
+            attrValue = ((Element) node).getAttribute(attributeName.getLocalPart());
+        }
+
         return node.getNodeType() == Node.ELEMENT_NODE
-                && matchValue(((Element) node).getAttribute(attributeName));
+                && matchValue(attrValue);
+
     }
 
     private boolean matchValue(final String testValue) {
@@ -312,7 +349,7 @@ public boolean find(final Predicate predicate) {
         }
 
         final QName qname = testStep.getTest().getName();
-        if (qname.getNameType() != ElementValue.ATTRIBUTE || !qname.equals(attribute)) {
+        if (qname.getNameType() != ElementValue.ATTRIBUTE || !qname.equals(attributeName)) {
             return false;
         }