Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-2601

[GH-750] Update to allow casting objects to strings (if possible) for non-composi...

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Invalid
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Security Level: All
    • Labels:
      None

      Description

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

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

      Message:

      ...te-key id attributes.

      Bug discovered when using a table with 2 columns, both being used as a composite primary key.
      Columns are:

      • user column linked by foreign key (which, when imported, was not mapped as an @id field)
      • datetime column

      Php Datetime class was extended to implement a __toString function (as described here http://stackoverflow.com/a/15085566), but this was not being called because doctrine was not recognising the entity as having a composite primary key (due to the non-datetime user key being joined via foreign key (which may in itself be a bug)).

      When adding an extra empty text field as part of the composite key, everything worked, it's just because the only non-joined-column was an object, and it managed to avoid the (implode) code that would have cast it as a string.

      This change casts the object, in this case a custom datetime, to a string, if it is not part of a composite key.

        Activity

        Doctrine Bot created issue -
        Benjamin Eberlei made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Doctrine Bot
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: