Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing success code for the SystemInfoDumpCommand #58

Merged
merged 2 commits into from
Sep 14, 2020

Conversation

konradoboza
Copy link
Member

@konradoboza konradoboza commented Sep 14, 2020

JIRA: https://jira.ez.no/browse/EZP-31879

Symfony 5 commands need to return an exit code that is currently missing. Due to that, the command results in error in eZ Platform 3.1:
Return value of "EzSystems\EzSupportToolsBundle\Command\SystemInfoDumpCommand::execute()" must be of the type int, "null" returned.

@micszo micszo self-assigned this Sep 14, 2020
@konradoboza konradoboza removed their assignment Sep 14, 2020
Copy link
Member

@micszo micszo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Approved on eZ Platform EE v3.1.2 with diff.

@micszo micszo removed their assignment Sep 14, 2020
@lserwatka lserwatka merged commit 85625d0 into 2.1 Sep 14, 2020
@lserwatka lserwatka deleted the command_success_code_missing branch September 14, 2020 11:31
@lserwatka
Copy link
Member

You can merge it up.

@konradoboza
Copy link
Member Author

Done.

@dabrt
Copy link
Contributor

dabrt commented Sep 14, 2020

Konrad, is there a Jira ticket for this change?

@konradoboza
Copy link
Member Author

No there is not. If you need one for the release notes though, I can create it right away. 😉

@lserwatka
Copy link
Member

We need one. We always need it for release notes.

@konradoboza
Copy link
Member Author

Here it is: https://jira.ez.no/browse/EZP-31879.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

8 participants