Problem/Motivation

error when install module on Drupal 10, PHP Fatal error: Type of Drupal\linkchecker\Commands\LinkCheckerCommands::$logger must be ?Psr\Log\LoggerInterface (as in class Drush\Commands\DrushCommands) in /app/docroot/modules/contrib/linkchecker/src/Commands/LinkCheckerCommands.php on line 16

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork linkchecker-3406749

Command icon 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:

About issue forks

Comments

abu-zakham created an issue. See original summary.

richardrobinson’s picture

richardrobinson commented

I'm running 10.1.5 and there were no issues upon install. Can you provide more details on your setups and steps to reproduce?

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented

That probably means it's not compatible with drush 12. Install drush 11 if possible for your project

chetan 11 made their first commit to this issue’s fork.

chetan 11 opened merge request !57

chetan 11’s picture

chetan 11 commented
Status: Active » Needs review

Hi,
I have fixed the above error on "link checker" module, please check the raised MR.
Thanks.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Status: Needs review » Reviewed & tested by the community

seems that this is needed for drush commands to work with LinkChecker on D10

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Assigned: Unassigned » eiriksm

  • eiriksm committed dc351eaf on 2.0.x authored by chetan 11 
    Issue #3406749 by abu-zakham, chetan 11, eiriksm: D10 PHP Fatal error
eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Assigned: eiriksm » Unassigned
Status: Reviewed & tested by the community » Fixed
eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented

Thanks everyone. Those changes make sense to me

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.