You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@stanbol.apache.org by rw...@apache.org on 2013/11/06 14:32:35 UTC
svn commit: r1539337 -
/stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java
Author: rwesten
Date: Wed Nov 6 13:32:35 2013
New Revision: 1539337
URL: http://svn.apache.org/r1539337
Log:
merged fix for STANBOL-1204 to the 0.12.0 releaseing branch
Modified:
stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java
Modified: stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java
URL: http://svn.apache.org/viewvc/stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java?rev=1539337&r1=1539336&r2=1539337&view=diff
==============================================================================
--- stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java (original)
+++ stanbol/branches/release-0.12/entityhub/query/sparql/src/main/java/org/apache/stanbol/entityhub/query/sparql/SparqlQueryUtils.java Wed Nov 6 13:32:35 2013
@@ -752,6 +752,9 @@ public final class SparqlQueryUtils {
if (constraint.getPatternType() == PatternType.none) {
if (constraint.isCaseSensitive()) {
boolean first = true;
+ if(constraint.getTexts().size() > 1){
+ queryString.append('('); //start language filter group (STANBOL-1204)
+ }
for (String textConstraint : constraint.getTexts()) {
if (first) {
first = false;
@@ -764,6 +767,9 @@ public final class SparqlQueryUtils {
queryString.append("\")");
}
}
+ if(constraint.getTexts().size() > 1){
+ queryString.append(')'); //end language filter group (STANBOL-1204)
+ }
} else {
Collection<String> regexQueryTexts = new ArrayList<String>(
constraint.getTexts().size());
@@ -795,7 +801,6 @@ public final class SparqlQueryUtils {
}
}
} // else nothing to do add language Filters
- // TODO check if FILTER ( is already written!
if (constraint.getLanguages() != null && !constraint.getLanguages().isEmpty()) {
log.trace("Constraint has languages [filter-added :: {}].",
@@ -916,6 +921,9 @@ public final class SparqlQueryUtils {
Collection<String> regexContraints,
boolean isCasesensitive) {
boolean first = true;
+ if(regexContraints.size() > 1){
+ queryString.append('('); //STANBOL-1204
+ }
for (String regex : regexContraints) {
if (regex != null && !regex.isEmpty()) {
if (first) {
@@ -932,6 +940,9 @@ public final class SparqlQueryUtils {
queryString.append(')');
}
}
+ if(regexContraints.size() > 1){
+ queryString.append(')'); //STANBOL-1204
+ }
}
/**