Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thedavidmeister’s picture

Status: Active » Needs review
FileSize
1.27 KB

patch.

c4rl’s picture

Title: Replace calls to theme() with renderable arrays in Seven theme » Replace theme() with drupal_render() in seven.theme

Retitling

thedavidmeister’s picture

Issue tags: +Novice, +Needs manual testing
thedavidmeister’s picture

this needs manual testing because i refactored things slightly.

Miroling’s picture

Status: Needs review » Reviewed & tested by the community

Manually tested at Home > Administration > People page. Path tested & working fine.

thedavidmeister’s picture

Issue tags: -Novice, -Needs manual testing
Miroling’s picture

Issue tags: +CodeSprintUA

Missed tag for manual testing comment.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Nice tidy up :)

Committed bc7e09a and pushed to 8.x. Thanks!

hass’s picture

Status: Fixed » Needs work

Strings need to be ucfirst.

kay_v’s picture

Assigned: Unassigned » kay_v
Status: Needs work » Needs review
Issue tags: -CodeSprintUA
FileSize
582 bytes

changing strings to ucfirst

kay_v’s picture

FileSize
582 bytes

oops - changing patch filename

scor’s picture

Status: Needs review » Reviewed & tested by the community

Simple string change. Looks good.

alexpott’s picture

Committed 6c5cc0c and pushed to 8.x. Thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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