Fix matching of records with predicate variables.

linkedfactory · Dec 17, 2024 · 5986330 · 5986330
1 parent aaebd58
commit 5986330
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...o.github.linkedfactory.core/src/main/java/io/github/linkedfactory/core/rdf4j/aas/AAS.java b/...o.github.linkedfactory.core/src/main/java/io/github/linkedfactory/core/rdf4j/aas/AAS.java
@@ -62,6 +62,8 @@ static Value toRdfValue(Object value, ValueFactory vf) {
 									Base64.getEncoder().encodeToString(idStr.getBytes(StandardCharsets.UTF_8)));
 						default:
 							// do not convert the reference to an IRI
+							// drop id string
+							idStr = null;
 					}
 				}
 			}

diff --git a/...ore/src/main/java/io/github/linkedfactory/core/rdf4j/aas/query/AasEvaluationStrategy.java b/...ore/src/main/java/io/github/linkedfactory/core/rdf4j/aas/query/AasEvaluationStrategy.java
@@ -91,11 +91,12 @@ public CloseableIteration<BindingSet, QueryEvaluationException> evaluate(Stateme
 
 				@Override
 				public boolean hasNext() throws QueryEvaluationException {
-					if (next == null && it.hasNext()) {
+					while (next == null && it.hasNext()) {
 						Record r = it.next();
 						Value newObjectValue = toRdfValue(r.getValue());
 						if (objectValue != null && !objectValue.equals(newObjectValue)) {
-							return false;
+							// try next value
+							continue;
 						}
 
 						CompositeBindingSet newBs = new CompositeBindingSet(bs);