You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by xristy <gi...@git.apache.org> on 2017/12/09 17:39:00 UTC

[GitHub] jena pull request #325: fix JENA-1439 graph queries fail 'lang:xx'

GitHub user xristy opened a pull request:

    https://github.com/apache/jena/pull/325

    fix JENA-1439  graph queries fail 'lang:xx'

    Fixes JENA-1439 graph queries fail to preserve text:query 'lang:xx' arg. TextQueryPF.extractArg(...) removed the arg unnecessarily from the input list, side-effecting the list which was saved in an OpPropFunc for repeated use on graph iterations and thus the argument was only present on the first graph in the iteration

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/BuddhistDigitalResourceCenter/jena JENA-1439-graphs

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jena/pull/325.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #325
    
----
commit 3d63b3d2d62a30690b7359703fc39aa05001be31
Author: Chris Tomlinson <ct...@moonvine.org>
Date:   2017-12-09T17:37:51Z

    fix JENA-1439  graph queries fail 'lang:xx'
    
    Fixes JENA-1439 graph queries fail to preserve text:query 'lang:xx' arg. TextQueryPF.extractArg(...) removed the arg unnecessarily from the input list, side-effecting the list which was saved in an OpPropFunc for repeated use on graph iterations and thus the argument was only present on the first graph in the iteration

----


---

[GitHub] jena issue #325: fix JENA-1439 graph queries fail 'lang:xx'

Posted by osma <gi...@git.apache.org>.
Github user osma commented on the issue:

    https://github.com/apache/jena/pull/325
  
    Great detective work @xristy ! It would be nice to have a unit test that checks this, though the fix seems rather obvious after finding out the culprit, and the bug unlikely to surface again.


---

[GitHub] jena pull request #325: fix JENA-1439 graph queries fail 'lang:xx'

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/jena/pull/325


---