OyvindLGjesdal opened a new pull request, #2121:
URL: https://github.com/apache/jena/pull/2121
GitHub issue resolved #2094
Pull request Description:
I'm not familiar with writing java tests, so not sure why the jena-text
doesn't pick up the example. However when running a single test from intellij,
it passes. The test could also be better named. Could I get some guiding on
this?
I see a similar fix with parens is also used in other paths in the code for
example in line 765 and 771 of the same file.
```
qString = "(" + qString + ") AND " + (!lang.equals("none") ?
langField+":"+lang : "-"+langField+":*");
log.trace("query$ ADDING LANG qString: {} ", qString) ;
}
```
```
if (graphURI != null) {
String escaped = QueryParserBase.escape(graphURI) ;
qString = "(" + qString + ") AND " + getDocDef().getGraphField()
+ ":" + escaped ;
}
```
----
- [x] Tests are included.
- [ ] Documentation change and updates are provided for the [Apache Jena
website](https://github.com/apache/jena-site/)
- [ ] Commits have been squashed to remove intermediate development commit
messages.
- [ ] Key commit messages start with the issue number (GH-xxxx, or if in
JIRA, JENA-xxxx)
By submitting this pull request, I acknowledge that I am making a
contribution to the Apache Software Foundation under the terms and conditions
of the [Contributor's
Agreement](https://www.apache.org/licenses/contributor-agreements.html).
----
See the [Apache Jena "Contributing"
guide](https://github.com/apache/jena/blob/main/CONTRIBUTING.md).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
