Doctrine 2 - ORM
  1. Doctrine 2 - ORM
  2. DDC-2273

[GH-566] Added missing return statement to AbstractCommand.

    Details

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

      Description

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

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

      Message:

      The `AbstractCommand::execute()` method should return an `integer`, or `null`.

      Although in PHP, omitting the `return` statement is equivalent to returning `null`, it is recommended to put an explicit `return` to clearly differentiate it from `return void`.

      I've added `return 0;` here, keeping the existent behavior.

        Activity

        Hide
        Benjamin Eberlei added a comment -

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

        Show
        Benjamin Eberlei added a comment - A related Github Pull-Request [GH-566] was closed https://github.com/doctrine/doctrine2/pull/566
        Show
        Fabio B. Silva added a comment - Merged : https://github.com/doctrine/doctrine2/commit/d3cbdfcafa3e8dbb1d3f7029f31eb6412855c097

          People

          • Assignee:
            Benjamin Eberlei
            Reporter:
            Benjamin Eberlei
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: