Problem/Motivation

At some point I got:

Error: Call to undefined method Drupal\pathauto\VerboseMessenger::addError()

web/modules/contrib/pathauto/src/PathautoGenerator.php:352
web/modules/contrib/pathauto/pathauto.module:91
web/core/lib/Drupal/Core/Extension/ModuleHandler.php:403
web/core/lib/Drupal/Core/Entity/EntityStorageBase.php:206
web/core/lib/Drupal/Core/Entity/ContentEntityStorageBase.php:756
web/core/lib/Drupal/Core/Entity/EntityStorageBase.php:507
web/core/lib/Drupal/Core/Entity/ContentEntityStorageBase.php:(641
(...)

Proposed resolution

Fix it.

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#9 3003373-9.patch4.35 KBidebr
#9 interdiff-2-9.txt3.97 KBidebr
#2 3003373-2.patch495 bytesclaudiu.cristea
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

claudiu.cristea created an issue. See original summary.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Issue summary: View changes
FileSize
3003373-2.patch495 bytes

Patch.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea as a volunteer and at Webikon for European Commission and European Union Institutions, Agencies and Bodies commented
Status: Needs work » Needs review
pookmish’s picture

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

Simple fix that got me back on track.

rp7’s picture

rp7 CreditAttribution: rp7 for Government of Flanders commented

Patch in #2 solved the issue for me as well.

el1_1el’s picture

el1_1el CreditAttribution: el1_1el commented

same here for #2. Thanks @claudiu.cristea

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

+1

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Needs work

The fix isn't correct, this is an unfortunate conflict between our own messenger service and the one from drupal core.

This needs to either inject the core messenger service as a differnt property and call addError() on that or rename the current $this->messenger and use the MessengerTrait.

idebr’s picture

idebr CreditAttribution: idebr at ezCompany commented
Status: Needs work » Needs review
FileSize
interdiff-2-9.txt3.97 KB
3003373-9.patch4.35 KB

Attached patch implements the following changes:

  1. $messenger property renamed to $pathautoMessenger to distinguish between the different messenger implementations.
  2. Updated the ->addError($e->getMessage()); method call to use the Drupal core Messenger service using the MessengerTrait

  • Berdir committed a5f8ce1 on 8.x-1.x authored by idebr 
    Issue #3003373 by idebr, claudiu.cristea, rp7, pookmish, el1_1el, Chris...
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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

marcelovani’s picture

marcelovani
Primary language Portuguese, Brazil
Location London
CreditAttribution: marcelovani at Dennis commented

Would be nice to have a release