In Drupal8 deprectaed methods will be removed before Drupal 9.
Replaced with drupal_render() method from the codebase.

CommentFileSizeAuthor
#4 interdiff-2719983-2-4.txt1.02 KBrajeshwari10
#4 2719983-4.patch1.02 KBrajeshwari10
admin_menu-drupalrender.patch1003 bytespashupathi nath gajawada

Comments

Pashupathi Nath G created an issue. See original summary.

Status: Needs review » Needs work

The last submitted patch, admin_menu-drupalrender.patch, failed testing.

pashupathi nath gajawada’s picture

pashupathi nath gajawada commented
Status: Needs work » Needs review
rajeshwari10’s picture

rajeshwari10 commented
StatusFileSize
new 2719983-4.patch1.02 KB
new interdiff-2719983-2-4.txt1.02 KB

Changed \Drupal::service($content) to \Drupal::service('renderer')->render($content).

Thanks!!

Status: Needs review » Needs work

The last submitted patch, 4: 2719983-4.patch, failed testing.

deepakkumar14’s picture

deepakkumar14 commented
Status: Needs work » Needs review
deepakkumar14’s picture

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

I have reviewed patch and its working fine for me patch is removing all the deprecated function drupal_render() from admin_menu module.

truls1502’s picture

truls1502
he/him
Primary language Norwegian Bokmål
commented
Status: Reviewed & tested by the community » Needs review

Let's set the status to NR so people look at this.

pifagor’s picture

pifagor commented

Looks good

pifagor’s picture

pifagor commented
Status: Needs review » Reviewed & tested by the community
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Thanks everyone!

  • thalles committed 597e10d on 8.x-3.x authored by rajeshwari10 
    Issue #2719983 by rajeshwari10, pashupathi nath gajawada, deepakkumar14...
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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