Replace deprecated usage of drupal_render with a theme_render_and_autoescape.

Comments

heykarthikwithu created an issue. See original summary.

heykarthikwithu’s picture

Assigned: heykarthikwithu » Unassigned
Status: Active » Needs review
StatusFileSize
new1.05 KB
gausarts’s picture

gausarts’s picture

Status: Needs review » Needs work

Please also remove those TODOs mentioned above any of the updated code. Thanks.

heykarthikwithu’s picture

Assigned: Unassigned » heykarthikwithu
heykarthikwithu’s picture

Assigned: heykarthikwithu » Unassigned
Status: Needs work » Needs review

@gausarts, Currenlty i don't see any other depreciated methods in the code base.

gausarts’s picture

Sorry for not being clear, I meant those lines saying:
// @todo #2575445 theme_render_and_autoescape()

gausarts’s picture

Status: Needs review » Needs work

This seems is going to be deprecated as well here ;)

gausarts’s picture

Title: Replace deprecated usage of drupal_render with a theme_render_and_autoescape. » Replace deprecated usage of drupal_render with renderer
Status: Needs work » Needs review
StatusFileSize
new2.74 KB

Let's fix this with the more future-proof replacement.

  • gausarts committed f838c64 on 8.x-1.x
    - #2662424 by heykarthikwithu, gausarts: Replace deprecated usage of...
gausarts’s picture

Status: Needs review » Fixed

Committed. Thank you for contribution.

Status: Fixed » Closed (fixed)

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