DDC-3717: [GH-1397] Add Expr::concat support for multiple arguments #4565

Closed
opened 2026-01-22 14:44:58 +01:00 by admin · 8 comments
Owner

Originally created by @doctrinebot on GitHub (Apr 28, 2015).

Originally assigned to: @guilhermeblanco on GitHub.

Jira issue originally created by user @doctrinebot:

This issue is created automatically through a Github pull request on behalf of giosh94mhz:

Url: https://github.com/doctrine/doctrine2/pull/1397

Message:

DQL CONCAT function support variable number of arguments (2+) since version 2.4.0.

The Expr class, on the other hand, is still limited to 2 arguments, and require multiple call to achive a similar result (see this SO question). I've improved Expr implementation to support variable number of arguments in a backward compatible way.

Please, consider backporting to 2.4 branch if you plan to release other versions, since 2.5 introduce many features and may not be possible to easily upgrade. I may be wrong, though.

Originally created by @doctrinebot on GitHub (Apr 28, 2015). Originally assigned to: @guilhermeblanco on GitHub. Jira issue originally created by user @doctrinebot: This issue is created automatically through a Github pull request on behalf of giosh94mhz: Url: https://github.com/doctrine/doctrine2/pull/1397 Message: DQL CONCAT function support variable number of arguments (2+) since version `2.4.0`. The `Expr` class, on the other hand, is still limited to 2 arguments, and require multiple call to achive a similar result (see [this SO question](http://stackoverflow.com/questions/22169324/concat-three-or-more-fields-in-doctrine/29913780)). I've improved `Expr` implementation to support variable number of arguments in a backward compatible way. Please, consider backporting to `2.4` branch if you plan to release other versions, since `2.5` introduce many features and may not be possible to easily upgrade. I may be wrong, though.
admin added the Improvement label 2026-01-22 14:44:58 +01:00
admin closed this issue 2026-01-22 14:44:58 +01:00
Author
Owner

@doctrinebot commented on GitHub (Sep 10, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was labeled:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Sep 10, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was labeled: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 6, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was assigned:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 6, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was assigned: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 6, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was unassigned:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 6, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was unassigned: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 6, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was unlabeled:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 6, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was unlabeled: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 7, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was merged:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 7, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was merged: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 7, 2015):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Nov 7, 2015): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Nov 7, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was unassigned:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 7, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was unassigned: https://github.com/doctrine/doctrine2/pull/1397
Author
Owner

@doctrinebot commented on GitHub (Nov 7, 2015):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-1397] was assigned:
https://github.com/doctrine/doctrine2/pull/1397

@doctrinebot commented on GitHub (Nov 7, 2015): Comment created by @doctrinebot: A related Github Pull-Request [GH-1397] was assigned: https://github.com/doctrine/doctrine2/pull/1397
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#4565