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

CommentFileSizeAuthor
#1 2009684-replace_theme_with_drupal_render_in_tour-1.patch1.21 KBaaronott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaronott’s picture

aaronott CreditAttribution: aaronott commented
Status: Active » Needs review
FileSize
2009684-replace_theme_with_drupal_render_in_tour-1.patch1.21 KB

Status: Needs review » Needs work

The last submitted patch, 2009684-replace_theme_with_drupal_render_in_tour-1.patch, failed testing.

helga.cheberakha’s picture

helga.cheberakha CreditAttribution: helga.cheberakha commented
Assigned: danylevskyi » helga.cheberakha

WDG (Ukraine,Kharkov) want to implement this on Code Sprint UA.

danylevskyi’s picture

danylevskyi
Location Myrhorod / Ukraine
CreditAttribution: danylevskyi commented
Assigned: Unassigned » danylevskyi

I'd like to take it. #2009672: [META] CodeSprint UA 2013

danylevskyi’s picture

danylevskyi
Location Myrhorod / Ukraine
CreditAttribution: danylevskyi commented
Assigned: helga.cheberakha » danylevskyi
Status: Needs work » Needs review

#1: 2009684-replace_theme_with_drupal_render_in_tour-1.patch queued for re-testing.

danylevskyi’s picture

danylevskyi
Location Myrhorod / Ukraine
CreditAttribution: danylevskyi commented
Assigned: danylevskyi » Unassigned
Status: Needs review » Reviewed & tested by the community
danylevskyi’s picture

danylevskyi
Location Myrhorod / Ukraine
CreditAttribution: danylevskyi commented
Issue tags: +CodeSprintUA
alexpott’s picture

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

Committed c39f366 and pushed to 8.x. Thanks!

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