Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
theme system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
20 Jan 2014 at 21:01 UTC
Updated:
29 Jul 2014 at 23:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
internetdevels commentedOnly one theme() cal has been found.
Comment #2
star-szrSince there is only one call I think we can take a cleaner approach, check how template_preprocess_page() does it:
Comment #3
star-szrComment #4
chakrapani commentedWorking on this..
Comment #5
chakrapani commentedHere we go!
Comment #6
jessebeach commentedThis is fine. Nearly the same code exists in
template_preprocess_pagefor rendering status messages.Comment #7
star-szrYeah this is fine but I think the drupal_render() is really unnecessary here. Just set $variables['messages'] the same way template_preprocess_page() does.
Comment #8
chakrapani commentedCottser, you are right, it does work without the drupal_render here.
Adding the new re-roll again.
Comment #9
star-szrThanks @chakrapani!
Comment #10
catchCommitted/pushed to 8.x, thanks!