Problem/Motivation

As seen in many other modules, the installer now utilizes the NullLogger which satisfies LoggerInterface but not LoggerChannelInterface, which is not deprecated but states it should not itself be used for typehinting.

Steps to reproduce

Crashed site on install in Drupal 10.3.

Proposed resolution

Adjust typehint.

Remaining tasks

Review and commit.

User interface changes

None.

API changes

None.

Data model changes

None.

Issue fork flysystem_s3-3493248

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

bradjones1 created an issue. See original summary.

bradjones1’s picture

bradjones1
Primary language English
commented
Related issues: +#3469460: NullLogger not implements LoggerChannelInterface

bradjones1 opened merge request !19

bradjones1’s picture

bradjones1
Primary language English
commented
Related issues: +#3493631: Fix GitLab CI (disable D11 testing for the moment)

  • bradjones1 committed 9de321a3 on 2.1.x 
    Issue #3493248 by bradjones1: Plugin logger must use `LoggerInterface`...
bradjones1’s picture

bradjones1
Primary language English
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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