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
Refactor commands to avoid wrong output #827
Conversation
like in CreateDatabaseDoctrineCommand
like in CreateDoctrineDatabaseCommand
before not output of the dropped database name was given, only the connection name.
@@ -49,7 +49,12 @@ protected function configure() | |||
*/ | |||
protected function execute(InputInterface $input, OutputInterface $output) | |||
{ | |||
$connection = $this->getDoctrineConnection($input->getOption('connection')); | |||
$connectionName = $input->getOption('connection'); | |||
if (empty($connectionName) === true) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
=== true
is useless. empty($connectionName)
is already a boolean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tbh I just copy pasted this part from the create command
shall I change this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i changed it 👍
I also fixed the Code-Quality build, which belongs to this PR. Shall I change them in this or in an extra PR? |
Would be happy to get this merged 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Thanks @OskarStark |
Please review commit by commit.
Could be squashed after review into one.
Before:
this is wrong, as my DB is
luna
and my connection isdefault
After: