Skip to content

drush core:status no longer works in Drush 11 with non-core db drivers#5023

Merged
weitzman merged 1 commit intodrush-ops:11.xfrom
mondrake:patch-1
Jan 17, 2022
Merged

drush core:status no longer works in Drush 11 with non-core db drivers#5023
weitzman merged 1 commit intodrush-ops:11.xfrom
mondrake:patch-1

Conversation

@mondrake
Copy link
Contributor

drush core:status worked in Drush 10 with a non-core db driver, but on Drush 11 it fails with

TypeError: Drush\Sql\SqlBase::getInstance(): Return value must be of type ?Drush\Sql\SqlBase, none returned in Drush\Sql\SqlBase::getInstance() (line 114 of /home/runner/work/drudbal/drudbal/vendor/drush/drush/src/Sql/SqlBase.php).

This seems due to introduced return typehinting in the getInstance() method. It is set to ?self but if the driver class is missing there is no return stament called hence the TypeError.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants