Doctrine 1
  1. Doctrine 1
  2. DC-1056

Doctrine is not compatible with PHP 5.4 due to change in serialize() behaviour.

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.2.3
    • Fix Version/s: None
    • Component/s: Record, Relations
    • Labels:
      None
    • Environment:
      PHP 5.4+

      Description

      In PHP 5.4 there is a change in the way the object references are serialized:

      Quote:
      "Support for object references in recursive serialize() calls
      Prior to PHP 5.4, object references where not saved in recursive serialize calls."

      This minor change, breaks down serialization of collections when column of type "array" is present - double serialization occurs.
      I'm attaching a patch fixing the issue.

        Activity

        Hide
        Colin Darie added a comment -

        I confirm for possible future readers: this patch works perfectly well. (cf github for several forks of doctrine with other bugfixes).

        Show
        Colin Darie added a comment - I confirm for possible future readers: this patch works perfectly well. (cf github for several forks of doctrine with other bugfixes).
        Hide
        steven added a comment -

        Hi all, does somebody knows where can I get a copy of the Doctrine 1.2.4 version but running on php 5.4?
        Thise version you're talking about

        Thanks

        Show
        steven added a comment - Hi all, does somebody knows where can I get a copy of the Doctrine 1.2.4 version but running on php 5.4? Thise version you're talking about Thanks
        Hide
        Marcin Gil added a comment -

        I sent you URL to our private svn repo.

        Show
        Marcin Gil added a comment - I sent you URL to our private svn repo.
        Hide
        steven added a comment -

        Thanks, you've saved mi life

        Show
        steven added a comment - Thanks, you've saved mi life
        Hide
        Marcin Gil added a comment -

        New, better patch.

        Show
        Marcin Gil added a comment - New, better patch.
        Hide
        Marcin Gil added a comment -

        Hi, I have uploaded a better patch which resolved one more issue.

        Show
        Marcin Gil added a comment - Hi, I have uploaded a better patch which resolved one more issue.
        Hide
        Christophe Coevoet added a comment -

        Please use a pull request on github to submit a patch

        Show
        Christophe Coevoet added a comment - Please use a pull request on github to submit a patch

          People

          • Assignee:
            Jonathan H. Wage
            Reporter:
            Marcin Gil
          • Votes:
            2 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated: