Details

    • Type: Bug Bug
    • Status: Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.3, 2.4.2
    • Fix Version/s: 2.3.3
    • Component/s: None
    • Security Level: All
    • Labels:
      None

      Description

      Column type json_array can be set to nullable, but if there's null in the database, it is returned as an empty array to PHP.

      Null should be returned instead, as that's how the other types behave too.

        Issue Links

          Activity

          Jan Hruban created issue -
          Hide
          Benjamin Eberlei added a comment -

          This was fixed in 2.3.3

          Show
          Benjamin Eberlei added a comment - This was fixed in 2.3.3
          Benjamin Eberlei made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 2.3.3 [ 10328 ]
          Resolution Fixed [ 1 ]
          Hide
          Joseph Wynn added a comment -

          I'm seeing this behaviour again in v2.5.0-BETA3 (6d0b048). If I get time this week I can perform a bisect to figure out when it regressed.

          Show
          Joseph Wynn added a comment - I'm seeing this behaviour again in v2.5.0-BETA3 (6d0b048). If I get time this week I can perform a bisect to figure out when it regressed.
          Hide
          Joseph Wynn added a comment -

          Actually I don't think this was a regression; it looks like a fix was never made. I've opened a PR: https://github.com/doctrine/dbal/pull/655

          Show
          Joseph Wynn added a comment - Actually I don't think this was a regression; it looks like a fix was never made. I've opened a PR: https://github.com/doctrine/dbal/pull/655
          Hide
          Doctrine Bot added a comment -

          A related Github Pull-Request [GH-655] was closed:
          https://github.com/doctrine/dbal/pull/655

          Show
          Doctrine Bot added a comment - A related Github Pull-Request [GH-655] was closed: https://github.com/doctrine/dbal/pull/655
          Marco Pivetta made changes -
          Link This issue is referenced by DBAL-966 [ DBAL-966 ]
          Hide
          Doctrine Bot added a comment -

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

          Show
          Doctrine Bot added a comment - A related Github Pull-Request [GH-655] was assigned: https://github.com/doctrine/dbal/pull/655
          Hide
          Marco Pivetta added a comment -

          Re-opened, as this behavior seems reproducible also in 2.4.x

          Show
          Marco Pivetta added a comment - Re-opened, as this behavior seems reproducible also in 2.4.x
          Marco Pivetta made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Assignee Benjamin Eberlei [ beberlei ] Marco Pivetta [ ocramius ]
          Marco Pivetta made changes -
          Affects Version/s 2.4.2 [ 10620 ]
          Hide
          Steve Müller added a comment -

          It was never fixed. That seems to have been a misunderstanding here. As pointed out by Marco Pivetta changing this behaviour is a BC break and can't be fixed before 3.0.

          Show
          Steve Müller added a comment - It was never fixed. That seems to have been a misunderstanding here. As pointed out by Marco Pivetta changing this behaviour is a BC break and can't be fixed before 3.0.

          This list may be incomplete, as errors occurred whilst retrieving source from linked applications:

          • Request to http://www.doctrine-project.org/fisheye/ failed: Error in remote call to 'FishEye 0 (http://www.doctrine-project.org/fisheye/)' (http://www.doctrine-project.org/fisheye) [AbstractRestCommand{path='/rest-service-fe/search-v1/crossRepositoryQuery', params={query=DBAL-446, expand=changesets[0:20].revisions[0:29],reviews}, methodType=GET}] : Received status code 503 (Service Temporarily Unavailable)

            People

            • Assignee:
              Marco Pivetta
              Reporter:
              Jan Hruban
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated: