DDC-2273: [GH-566] Added missing return statement to AbstractCommand. #2855

Closed
opened 2026-01-22 14:05:51 +01:00 by admin · 4 comments
Owner

Originally created by @doctrinebot on GitHub (Feb 3, 2013).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user @beberlei:

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.

Originally created by @doctrinebot on GitHub (Feb 3, 2013). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user @beberlei: 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.
admin added the Bug label 2026-01-22 14:05:51 +01:00
admin closed this issue 2026-01-22 14:05:52 +01:00
Author
Owner

@doctrinebot commented on GitHub (Feb 3, 2013):

Comment created by @beberlei:

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

@doctrinebot commented on GitHub (Feb 3, 2013): Comment created by @beberlei: A related Github Pull-Request [GH-566] was closed https://github.com/doctrine/doctrine2/pull/566
Author
Owner

@doctrinebot commented on GitHub (Feb 5, 2013):

Comment created by @FabioBatSilva:

Merged : d3cbdfcafa

@doctrinebot commented on GitHub (Feb 5, 2013): Comment created by @FabioBatSilva: Merged : https://github.com/doctrine/doctrine2/commit/d3cbdfcafa3e8dbb1d3f7029f31eb6412855c097
Author
Owner

@doctrinebot commented on GitHub (Feb 5, 2013):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Feb 5, 2013): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Apr 22, 2014):

Comment created by @doctrinebot:

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

@doctrinebot commented on GitHub (Apr 22, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-566] was closed: https://github.com/doctrine/dbal/pull/566
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#2855