Problem/Motivation

After enabling the module, drush throws this error: "Type of Drupal\save_all_nodes\Commands\SaveAllNodesCommands::$logger must be ?Psr\Log\LoggerInterface".

Steps to reproduce

Having installed:
1. Drupal 10.0.9
2. Drush 11.5.1
3. drupal/save_all_modules 1.0.1

Install the module with composer and try to using drush.

Proposed resolution

Fix the type hinting for variable $logger in /src/Commands/SaveAllNodesCommands.php on line 16

I have added a patch.

CommentFileSizeAuthor
save_all_nodes.1.x-dev.drush-logger-interface.patch711 bytesXalabama

Issue fork save_all_nodes-3364951

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

xalabama created an issue. See original summary.

geoanders’s picture

geoanders commented
Status: Needs work » Needs review

Zarpele made their first commit to this issue’s fork.

Zarpele opened merge request !2

zarpele’s picture

zarpele commented
Status: Needs review » Reviewed & tested by the community

I can confirm the patch solves the issue, I have created the MR of it.

geoanders’s picture

geoanders commented
Status: Reviewed & tested by the community » Fixed

Thanks for the fix! Now merged in.

Status: Fixed » Closed (fixed)

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