drupal_set_message() has been deprecated in favor of the messenger service.

CommentFileSizeAuthor
#5 protected_pages_deprecated_drupal_set_message_2956265_5.patch10.73 KBoksana-c
#2 protected_pages_deprecated_drupal_set_message_2956265_2.patch11.19 KBnkoporec

Comments

nkoporec created an issue. See original summary.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Assigned: nkoporec » Unassigned
Status: Active » Needs review
StatusFileSize
new protected_pages_deprecated_drupal_set_message_2956265_2.patch11.19 KB

Created a patch which replaces all the drupal_set_messege functions.

useernamee’s picture

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

Patch applies without problems. All occurrences of drupal_set_message() are gone.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: protected_pages_deprecated_drupal_set_message_2956265_2.patch, failed testing. View results

oksana-c’s picture

oksana-c commented
StatusFileSize
new protected_pages_deprecated_drupal_set_message_2956265_5.patch10.73 KB

rerolled for the latest dev branch

oksana-c’s picture

oksana-c commented
Version: 8.x-1.0 » 8.x-1.x-dev

  • oksana-c committed 62cd395 on 8.x-1.x 
    Issue #2956265 by nkoporec, oksana-c, useernamee: Replace usages of the...
oksana-c’s picture

oksana-c commented
Status: Needs work » Fixed

Committed. Thank you @nkoporec, @useernamee.

Status: Fixed » Closed (fixed)

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

chris matthews’s picture

chris matthews commented
Parent issue: » #3081271: Plan for Protected Pages 8.x-1.2 release