Problem/Motivation

Error on Monolog extra processor:

wodby@php.container:/var/www/html $ drush updb
[19-Jan-2024 07:19:47 UTC] PHP Fatal error:  Declaration of Drupal\monolog_extra\Logger\Processor\TruncateMessagesProcessor::__invoke(array $record) must be compatible with Monolog\Processor\ProcessorInterface::__invoke(Monolog\LogRecord $record) in /var/www/html/docroot/modules/contrib/monolog_extra/src/Logger/Processor/TruncateMessagesProcessor.php on line 35

Fatal error: Declaration of Drupal\monolog_extra\Logger\Processor\TruncateMessagesProcessor::__invoke(array $record) must be compatible with Monolog\Processor\ProcessorInterface::__invoke(Monolog\LogRecord $record) in /var/www/html/docroot/modules/contrib/monolog_extra/src/Logger/Processor/TruncateMessagesProcessor.php on line 35
 [warning] Drush command terminated abnormally.

Proposed resolution

Monolog invoke method should use LogRecord instead for array https://github.com/Seldaek/monolog/blob/main/UPGRADE.md

Issue fork monolog_extra-3415774

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

Eduardo Morales Alberti created an issue. See original summary.

Eduardo Morales Alberti opened merge request !3

eduardo morales alberti’s picture

eduardo morales alberti
Primary language Spanish
Location Spain, 🇪🇺
commented
Status: Active » Needs work
omarlopesino’s picture

omarlopesino
he/him
commented
Status: Needs work » Needs review
eduardo morales alberti’s picture

eduardo morales alberti
Primary language Spanish
Location Spain, 🇪🇺
commented

Covered with tests, ready to review

omarlopesino’s picture

omarlopesino
he/him
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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