Skip to content

T16599 pg groupby broken#16940

Merged
niden merged 3 commits into5.0.xfrom
T16599-pg-groupby-broken
Apr 30, 2026
Merged

T16599 pg groupby broken#16940
niden merged 3 commits into5.0.xfrom
T16599-pg-groupby-broken

Conversation

@niden
Copy link
Copy Markdown
Member

@niden niden commented Apr 30, 2026

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

Fixed Phalcon\Mvc\Model\Query\Builder::autoescape() incorrectly wrapping function expressions (e.g. DATE_PART(...)) in brackets when used in groupBy(), causing a "Column does not belong to any of the selected models" exception

Thanks

@niden niden self-assigned this Apr 30, 2026
@niden niden added bug A bug report status: medium Medium 5.0 The issues we want to solve in the 5.0 release labels Apr 30, 2026
@niden niden merged commit 563502c into 5.0.x Apr 30, 2026
150 checks passed
@niden niden deleted the T16599-pg-groupby-broken branch April 30, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5.0 The issues we want to solve in the 5.0 release bug A bug report status: medium Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant