Came across a bug in Drupal HEAD where template_preprocess_page() sets $variables['mission'] then clears it again thanks to some faulty logic. Thus, mission statements cannot be displayed.

This one-liner solves the problem for me.

ttyl!
--Alex Markley

CommentFileSizeAuthor
#2 theme_mission-177421-2.patch1.08 KBchx
#1 theme-mission-01.patch1.98 KBdesbeers
theme_mission_statement.diff673 bytesalexmarkley
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

desbeers’s picture

desbeers CreditAttribution: desbeers commented
FileSize
theme-mission-01.patch1.98 KB

Confirmed the problem and patch works well. Problem appeared in http://drupal.org/node/137236. I altered the patch a bit: changed the comment above the affected code, the previous one made no sense, and put the 'mission' code together in a more logical place.

chx’s picture

chx CreditAttribution: chx commented
FileSize
theme_mission-177421-2.patch1.08 KB

I shortened the patch.

desbeers’s picture

desbeers CreditAttribution: desbeers commented

Chx, yes, patch is shorter, but I liked to have all the 'mission' code grouped together. Just cosmetic; but easier to read.

dvessel’s picture

dvessel CreditAttribution: dvessel commented
Status: Needs review » Reviewed & tested by the community

#2 works for me. Lets move on.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Indeed, we try to keep variables together, so I committed #2. Thanks!

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)