Skip to content

Commit 97731a1

Browse files
committed
GH-4923 Spaces inserted by PropertyPathBuilder cause problems with AllegroGraph
- Removed spaces after prefix operators and before postfix operators
1 parent 23f3e8c commit 97731a1

File tree

6 files changed

+8
-7
lines changed

6 files changed

+8
-7
lines changed

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/InversePath.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@ public InversePath(PropertyPath path) {
2222
this.path = path;
2323
}
2424

25+
@Override
2526
public String getQueryString() {
26-
return "^ " + path.getQueryString();
27+
return "^" + path.getQueryString();
2728
}
2829
}

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/InversePredicatePath.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,6 @@ public InversePredicatePath(IRI predicate) {
3333

3434
@Override
3535
public String getQueryString() {
36-
return "^ " + predicate.getQueryString();
36+
return "^" + predicate.getQueryString();
3737
}
3838
}

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/NegatedPropertySet.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,12 @@ public NegatedPropertySet(PredicatePathOrInversePredicatePath... properties) {
3030
@Override
3131
public String getQueryString() {
3232
if (properties.length == 1) {
33-
return "! " + properties[0].getQueryString();
33+
return "!" + properties[0].getQueryString();
3434
} else {
3535
return Arrays
3636
.stream(properties)
3737
.map(QueryElement::getQueryString)
38-
.collect(Collectors.joining(" | ", "! ( ", " )"));
38+
.collect(Collectors.joining(" | ", "!( ", " )"));
3939
}
4040
}
4141
}

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/OneOrMorePath.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,6 @@ public OneOrMorePath(PropertyPath path) {
2424

2525
@Override
2626
public String getQueryString() {
27-
return path.getQueryString() + " +";
27+
return path.getQueryString() + "+";
2828
}
2929
}

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/ZeroOrMorePath.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,6 @@ public ZeroOrMorePath(PropertyPath path) {
2424

2525
@Override
2626
public String getQueryString() {
27-
return path.getQueryString() + " *";
27+
return path.getQueryString() + "*";
2828
}
2929
}

core/sparqlbuilder/src/main/java/org/eclipse/rdf4j/sparqlbuilder/constraint/propertypath/ZeroOrOnePath.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,6 @@ public ZeroOrOnePath(PropertyPath path) {
2424

2525
@Override
2626
public String getQueryString() {
27-
return path.getQueryString() + " ?";
27+
return path.getQueryString() + "?";
2828
}
2929
}

0 commit comments

Comments
 (0)