Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Type of Drupal\checklistapi\Commands\ChecklistapiCommands::$logger must be ?Psr\Log\LoggerInterface (as in class Drush\Commands\DrushCommands)
Steps to reproduce
- Install the module in D10 instance
- Run drush
Issue fork checklistapi-3310067
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
Lal_Comment #4
nkoporecThe issue is fixed, setting as RTBC.
Comment #5
jcnventura CreditAttribution: jcnventura at 1xINTERNET commentedYes, this fixes it.. But not correctly. Better drop the logger attribute completely and use $this->logger() instead, like Drush itself does.
Comment #6
jcnventura CreditAttribution: jcnventura at 1xINTERNET commentedComment #7
jcnventura CreditAttribution: jcnventura at 1xINTERNET commentedSee #3278542: Update $logger definition to be compatible with Drupal 10 where @nkoporec did a 2nd commit very similar to my 2nd commit here.
Comment #8
nkoporecLooks good to me, setting as RTBC.
Comment #11
TravisCarden CreditAttribution: TravisCarden at Acquia commentedMerged. Thanks, everyone!