[infinispan-issues] [JBoss JIRA] (ISPN-3442) Query API DSL returns bad result on calling negated between() with includeUpper/Lower(false)

[Adrian Nistor (JIRA)]

     [ https://issues.jboss.org/browse/ISPN-3442?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Nistor updated ISPN-3442:
--------------------------------

        Status: Resolved  (was: Pull Request Sent)
      Assignee: Jiří Holuša  (was: Sanne Grinovero)
    Resolution: Done


Integrated in master. Thanks!
                
> Query API DSL returns bad result on calling negated between() with includeUpper/Lower(false)
> --------------------------------------------------------------------------------------------
>
>                 Key: ISPN-3442
>                 URL: https://issues.jboss.org/browse/ISPN-3442
>             Project: Infinispan
>          Issue Type: Bug
>          Components: Querying
>            Reporter: Jiří Holuša
>            Assignee: Jiří Holuša
>
> Assuming existing of three users with IDs 1,2 and 3.
> Then this code's result is incorrect.
> {code:border=solid}
> Query q = queryFactory.from(User.class)
>             .not().having("id").between(1, 2).includeLower(false)
>             .toBuilder().build();
> {code}
> It returns no user whereas it should return two users with IDs 1 and 3. I debugged the org.infinispan.query.dsl.impl.JPAQueryGeneratorVisitor and found the problem in visit(BetweenOperator operator). 
> The code correctly changes the </<= and >/>= operators if between operator is negated, but forgets to change "AND" to "OR". 
> Thus the generated string query ends up like "id <= 1 AND id > 2" which is of course always empty result.
> The pull request contains fix to this bug and also extended tests for Query API DSL to increase code coverage.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira