Skip to content

Document FunctionsBuilder::stringAgg()#8279

Open
dereuromark wants to merge 3 commits into5.nextfrom
docs-string-agg
Open

Document FunctionsBuilder::stringAgg()#8279
dereuromark wants to merge 3 commits into5.nextfrom
docs-string-agg

Conversation

@dereuromark
Copy link
Copy Markdown
Member

@dereuromark dereuromark commented Apr 11, 2026

Summary

Documents the new portable FunctionsBuilder::stringAgg() method added in
cakephp/cakephp#19386.

  • Adds stringAgg() to the function list in the Query Builder page
  • Adds a "String Aggregation" subsection with an example and ordering argument
  • Notes per-driver translation (STRING_AGG / GROUP_CONCAT) and version
    support for aggregate-local ordering
  • Adds an entry to the 5.4 migration guide under Database

Adds documentation for the new portable stringAgg() method, covering
per-driver translation and supported ordering.
@dereuromark dereuromark marked this pull request as ready for review April 14, 2026 13:22
@dereuromark dereuromark added this to the 5.next milestone Apr 14, 2026
# Conflicts:
#	docs/en/appendices/5-4-migration-guide.md
Comment thread docs/en/orm/query-builder.md Outdated
`stringAgg()`
Aggregate string values using a separator. Translates to `STRING_AGG()`,
`GROUP_CONCAT()`, or `LISTAGG()` depending on the database driver.
`Assumes the first argument is a literal value.`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this formatted as code? Wouldn't bold be more appropriate?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of them on this page already are like this.

I assume the rst to md conversion did that. We will have to go through all pages and check where this was messed up by the script then :)

Copy link
Copy Markdown
Member Author

@dereuromark dereuromark Apr 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I told my virtual house slave to clean up. Looks about right.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, as I thought: 5.x is also already affected, cleanup PR: #8283

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants