Problem/Motivation

Advanced text formatter call a non exisiting method
\Drupal::messenger()->addNotice when using php filter to warn user and causes a PHP error

Proposed resolution

replace addNotice by addWarning ? here is a patch that does that

CommentFileSizeAuthor
#3 3231760-addNotice-method-does-not-exist-3.patch1.36 KBopi
fix-add-notice-non-existing-method.patch1.37 KBthony

Comments

Thony created an issue. See original summary.

carsteng’s picture

carsteng commented
Status: Active » Needs work

Do not call \Drupal ...

INstead use $this->messenger()->addWarning()

opi’s picture

opi
Primary language French
commented
StatusFileSize
new 3231760-addNotice-method-does-not-exist-3.patch1.36 KB

Here is a proper patch using $this->messenger()->addWarning method

opi’s picture

opi
Primary language French
commented
Status: Needs work » Needs review
bohus ulrych’s picture

bohus ulrych
Location Pilsen (Czechia)
commented

Thanks. This patch works - advanced_text_formatter 2.1.0 & Drupal 9.2.4

carsteng’s picture

carsteng commented
Status: Needs review » Reviewed & tested by the community
rajab natshah’s picture

rajab natshah
he/him
commented

Tested #3 in Drupal 8 and Drupal 9
Working ... Thank you :)

  • azovsky committed b25a425 on 2.1.x 
    Issue #3231760 by opi: addNotice Method does not exist
azovsky’s picture

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

Status: Fixed » Closed (fixed)

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