Remove deprecated drupal_get_message() drupal_set_message() from bootstrap.inc

Release notes snippet

drupal_set_message() has been removed. See https://www.drupal.org/node/2774931

CommentFileSizeAuthor
#2 3094903-2.patch6.26 KBkim.pepper

Comments

kim.pepper created an issue. See original summary.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Active » Needs review
StatusFileSize
new 3094903-2.patch6.26 KB

Initial patch.

dinesh18’s picture

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

Manually reviewed the patch. Looks good to me.
+1 to RTBC

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Small patch, but end of an era.

Here's the commit where drupal_set_message() was added:

commit 4e2c0b250e4f080d0bc3c4b587c6b2c53a0db7ac
Author: Dries Buytaert <dries@buytaert.net>
Date:   Mon Dec 1 13:45:33 2003 +0000

    - Introduced a drupal_set_message() and drupal_get_message() function.
    
      Contributed themes and modules need to be updated:
       - modules: status() is no more; use drupal_set_message() instead.
       - themes: use drupal_get_message() to check for status messages and
         visualize them.

Committed 2dd2d36 and pushed to 9.0.x. Thanks!

  • catch committed 2dd2d36 on 9.0.x 
    Issue #3094903 by kim.pepper, Dinesh18: Remove deprecated...
catch’s picture

catch
he/him
Primary language English
commented
Issue summary: View changes
Issue tags: +9.0.0 release notes

Adding a release notes snippet since this is used all over contrib and custom code we might as well specifically mention it.

Status: Fixed » Closed (fixed)

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