Problem/Motivation

Remove all the occurrence of drupal_set_message() is deprecated

Proposed resolution

Drupal 8.5.x has introduced a new messenger service.

recently drupal_set_message() has been deprecated
#2774931

CommentFileSizeAuthor
#4 devel-3032580-4.patch10.47 KBlussoluca
#2 removed_deprecated_drupal_set_message-3032580-2.patch9.44 KBakshay_d
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

akshay_d created an issue. See original summary.

akshay_d’s picture

akshay_d
Primary language Kannada
Location Bangalore
CreditAttribution: akshay_d at Specbee commented
Status: Active » Needs review
FileSize
removed_deprecated_drupal_set_message-3032580-2.patch9.44 KB

Removed all the occurrence of drupal_set_message()

akshay_d’s picture

akshay_d
Primary language Kannada
Location Bangalore
CreditAttribution: akshay_d at Specbee commented

please review

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca at H2H Creative Productions commented
FileSize
devel-3032580-4.patch10.47 KB

We also need to use MessengerTrait where not in ControllerBase or FormBase subclasses

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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

jonathan1055’s picture

jonathan1055 CreditAttribution: jonathan1055 as a volunteer commented
Title: drupal_set_message() is deprecated » Replace drupal_set_message() with $this->messenger in existing form/controller classes
Parent issue: » #3042575: [META] Deprecation issues in devel module

Added parent