Part of #2006152: [meta] Don't call theme() directly anywhere outside drupal_render().

CommentFileSizeAuthor
#4 locale-3.png86.23 KBjessebeach
#3 interdiff-2177667-1-3.txt681 bytesinternetdevels
#3 drupal-replace-theme-with-drupal-render-in-locale_bulk_inc-2177667-3.patch1017 bytesinternetdevels
#1 drupal-replace-theme-drupal-render-localebulkinc-2177667-1.patch1.03 KBinternetdevels

Comments

internetdevels’s picture

internetdevels commented
Status: Active » Needs review
StatusFileSize
new drupal-replace-theme-drupal-render-localebulkinc-2177667-1.patch1.03 KB

Patch attached.

Status: Needs review » Needs work

The last submitted patch, 1: drupal-replace-theme-drupal-render-localebulkinc-2177667-1.patch, failed testing.

internetdevels’s picture

internetdevels commented
Status: Needs work » Needs review
StatusFileSize
new drupal-replace-theme-with-drupal-render-in-locale_bulk_inc-2177667-3.patch1017 bytes
new interdiff-2177667-1-3.txt681 bytes

New try.

jessebeach’s picture

jessebeach commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new locale-3.png86.23 KB

A simple replacement of theme() with drupal_render().

This patch can be tested at the following path after enabling the locale module: /admin/config/regional/translate/import

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed e35f411 and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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