[PR #8087] Updated CONCAT to support SimpleArithmeticExpression #10803

New Issue

admin · 2026-01-22T16:08:44+01:00

admin commented

2026-01-22 16:08:44 +01:00

Original Pull Request: https://github.com/doctrine/orm/pull/8087

State: closed
Merged: No

While working on a project noticed that CONCAT does not support any expressions at the moment it only supportsStringPrimary

P.S. Decided not to rename/remove $firstStringPrimary and $secondStringPrimary because they are public... while they seem as useless fields.

My CONCAT example

    CONCAT(
        i.price
        *
        IFELSE(
          i.priceOrigin > 0.01,
          1,
          IFELSE(
            dd.value > 0 AND dd.value != 1,
            dd.value,
            1
          
        )
      ),
      ' EUR'
    ) AS price,

**Original Pull Request:** https://github.com/doctrine/orm/pull/8087 **State:** closed **Merged:** No --- While working on a project noticed that `CONCAT` does not support any expressions at the moment it only supports`StringPrimary` P.S. Decided not to rename/remove `$firstStringPrimary` and `$secondStringPrimary` because they are public... while they seem as useless fields. My CONCAT example ```dql CONCAT( i.price * IFELSE( i.priceOrigin > 0.01, 1, IFELSE( dd.value > 0 AND dd.value != 1, dd.value, 1 ) ), ' EUR' ) AS price, ```

admin added the pull-request label 2026-01-22 16:08:44 +01:00

admin referenced this issue

2026-01-22 16:14:40 +01:00

[PR #10803] Fix unserialize() errors when running tests on PHP 8.3 #12627

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: doctrine/archived-orm#10803