Doctrine 2 - ORM

Paginator: ResultVariable cannot be used in HavingClause

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: Git Master
  • Fix Version/s: 2.3
  • Component/s: ORM
  • Security Level: All
  • Labels:
    None

Description

Using the QueryBuilder, I'm getting the message

Notice: Undefined index: distance in vendor\doctrine\lib\Doctrine\ORM\Query\SqlWalker.php line 2128

when trying to add a "having" clause for the field "distance", which is defined in the "select" part of the statement:

$queryBuilder
    ->select('myEntity, MY_FUNCTION() AS distance')
    ->having('distance <= 10')
;

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabio B. Silva added a comment -

Christian, I couldn't reproduce.

Can you give more details ?

Thanks.

Show
Fabio B. Silva added a comment - Christian, I couldn't reproduce. Can you give more details ? Thanks.
Hide
Permalink
Christian Raue added a comment -

I tried it again and found out that it could also be an issue in https://github.com/whiteoctober/Pagerfanta rather in Doctrine itself because my query is passed to that pager to show a sliced result. Maybe that issue could already be resolved by https://github.com/whiteoctober/Pagerfanta/pull/46. Will give it a try...

Show
Christian Raue added a comment - I tried it again and found out that it could also be an issue in https://github.com/whiteoctober/Pagerfanta rather in Doctrine itself because my query is passed to that pager to show a sliced result. Maybe that issue could already be resolved by https://github.com/whiteoctober/Pagerfanta/pull/46 . Will give it a try...
Hide
Permalink
Christian Raue added a comment -

In fact, applying that patch made the query work. So this is not a Doctrine issue.

Show
Christian Raue added a comment - In fact, applying that patch made the query work. So this is not a Doctrine issue.
Hide
Permalink
Christian Raue added a comment -

Since the PR mentioned was not merged and Doctrine's pagination is used now, the issue still exists and I cannot use a result variable in a "having" clause. But I'd like to.

Show
Christian Raue added a comment - Since the PR mentioned was not merged and Doctrine's pagination is used now, the issue still exists and I cannot use a result variable in a "having" clause. But I'd like to.
Hide
Permalink
Christophe Coevoet added a comment -

This ticket is indeed invalid as it is talking about the query builder. Please open a dedicated ticket for the pagination issue

Show
Christophe Coevoet added a comment - This ticket is indeed invalid as it is talking about the query builder. Please open a dedicated ticket for the pagination issue
Hide
Permalink
Christian Raue added a comment -

But I am using the QueryBuilder and then passing the query to a DoctrineORMAdapter instance. The error message didn't change, just the line number:

Notice: Undefined index: distance in vendor\doctrine\lib\Doctrine\ORM\Query\SqlWalker.php line 2142

But I can confirm that this error doesn't occur when not using a paginator. How to continue?

Show
Christian Raue added a comment - But I am using the QueryBuilder and then passing the query to a DoctrineORMAdapter instance. The error message didn't change, just the line number: Notice: Undefined index: distance in vendor\doctrine\lib\Doctrine\ORM\Query\SqlWalker.php line 2142 But I can confirm that this error doesn't occur when not using a paginator. How to continue?
Hide
Permalink
Benjamin Eberlei added a comment -

Which paginator are you using?

Show
Benjamin Eberlei added a comment - Which paginator are you using?
Hide
Permalink
Christian Raue added a comment - - edited

Still https://github.com/whiteoctober/Pagerfanta/

Show
Christian Raue added a comment - - edited Still https://github.com/whiteoctober/Pagerfanta/
Hide
Permalink
Benjamin Eberlei added a comment -

Renamed and assigned issue.

Show
Benjamin Eberlei added a comment - Renamed and assigned issue.
Hide
Permalink
Alexander added a comment -

Fixed as sander was kind enough to resubmit the code to the doctrine2 repository now instead:
https://github.com/doctrine/doctrine2/pull/298

Show
Alexander added a comment - Fixed as sander was kind enough to resubmit the code to the doctrine2 repository now instead: https://github.com/doctrine/doctrine2/pull/298

People

  • Assignee:
    Benjamin Eberlei
    Reporter:
    Christian Raue
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: