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
[TASK] Migrate DatabaseCommandController to Symfony Commands #840
Conversation
b029faf
to
55ba84b
Compare
55ba84b
to
ae5f2ee
Compare
ae5f2ee
to
80f1617
Compare
Thanks for creating all these pull requests 😍
I'd like to do deprecation removal and streamlining a different task. Thanks! |
80f1617
to
2a2676d
Compare
consts are removed now, restore of removal of deprecations still pending... |
2a2676d
to
e88bc46
Compare
Removals are restored now |
e88bc46
to
ecddd47
Compare
ecddd47
to
d2b634a
Compare
|
||
$objectManager = GeneralUtility::makeInstance(ObjectManager::class); | ||
$this->connectionConfiguration = $connectionConfiguration | ||
?? $objectManager->get(ConnectionConfiguration::class); |
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.
using the object manager here is unnecessary and counter productive.
please keep using new, like in DatabaseExportCommand
|
||
class DatabaseExportCommand extends AbstractConvertedCommand |
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.
please keep this already converted command unchanged. thanks.
The following commands are migrated from a CommandController structure to Symfony Commands: - database:export - database:import - database:updateschema
The following commands are migrated from a CommandController structure
to Symfony Commands: