You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/06/01 20:49:04 UTC

[jira] [Commented] (JENA-1355) Wrong determination of variables with property functions.

    [ https://issues.apache.org/jira/browse/JENA-1355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16033656#comment-16033656 ] 

ASF GitHub Bot commented on JENA-1355:
--------------------------------------

GitHub user afs opened a pull request:

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

    JENA-1355: Fix to VarFinder for OpPropFunc

    

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

    $ git pull https://github.com/afs/jena jena1355-varfinder

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

    https://github.com/apache/jena/pull/258.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 #258
    
----
commit 688006bf0318c206fc16eba2a8a97239e17060ce
Author: Andy Seaborne <an...@apache.org>
Date:   2017-06-01T20:41:36Z

    JENA-1355: Fix to VarFinder for OpPropFunc

----


> Wrong determination of variables with property functions.
> ---------------------------------------------------------
>
>                 Key: JENA-1355
>                 URL: https://issues.apache.org/jira/browse/JENA-1355
>             Project: Apache Jena
>          Issue Type: Bug
>    Affects Versions: Jena 3.3.0
>            Reporter: Andy Seaborne
>            Assignee: Andy Seaborne
>
> Bug in {{VarFinder}}:
> {noformat}
> public static void main(String... args) {
>     String z = String.join("\n"
>                            ,"    (propfunc <http://example/pf>"
>                            ,"      ?x (?y 1)"
>                            ,"      (table unit))"
>                            );
>     vfind(z);                         
> }
> public static void vfind(String z) {
>     Op op = SSE.parseOp(z);
>     System.out.println(op);
>     VarFinder vf = VarFinder.process(op);
>     System.out.println(vf);
> }
> {noformat}
> prints
> {noformat}
> (propfunc <http://example/pf>
>   ?x (?y 1)
>   (table unit))
> Fixed:[?y], Filter:[], Filter2:[], Opt:[], Assign:[]
> {noformat}
> should be
> {noformat}
> Fixed:[?x, ?y], Filter:[], Filter2:[], Opt:[], Assign:[]
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)