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

CommentFileSizeAuthor
#1 2009686-replace_theme_with_drupal_render_in_tracker-1.patch1.23 KBaaronott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aaronott’s picture

aaronott CreditAttribution: aaronott commented
Status: Active » Needs review
FileSize
2009686-replace_theme_with_drupal_render_in_tracker-1.patch1.23 KB

Status: Needs review » Needs work

The last submitted patch, 2009686-replace_theme_with_drupal_render_in_tracker-1.patch, failed testing.

E.Kurko’s picture

E.Kurko CreditAttribution: E.Kurko commented
Assigned: Unassigned » E.Kurko
E.Kurko’s picture

E.Kurko CreditAttribution: E.Kurko commented
Issue tags: +CodeSprintUA

Tagged as CodeSprintUA

E.Kurko’s picture

E.Kurko CreditAttribution: E.Kurko commented
Status: Needs work » Needs review

#1: 2009686-replace_theme_with_drupal_render_in_tracker-1.patch queued for re-testing.

E.Kurko’s picture

E.Kurko CreditAttribution: E.Kurko commented
Status: Needs review » Reviewed & tested by the community

Looks like random test error. All test locally run good.

alexpott’s picture

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

Committed 0294c55 and pushed to 8.x. Thanks!

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