Using $variables['template_files'] in issue_cockpit.inc's template_preprocess_project_issue_issue_cockpit() kind of breaks the automatic suggestions for themes. Since it gets a higher priority than what the theme engine finds in themes directories.

It's possible for theme's to define the preprocess and change that, but the patch attached makes this unnecessary.

CommentFileSizeAuthor
project_issue-template_path-01.patch1.15 KBAmrMostafa

Comments

AmrMostafa’s picture

AmrMostafa commented
Priority: Normal » Minor
dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Sweet -- I didn't know you could do it that way. At the SF d.o redesign sprint when we were working on this, I asked folks the right way to do this and the hack of defining the variable in the preprocess function was suggested by josh_k (and not shot down by drumm and others). ;) Since it worked, I went with that. However, this approach is clearly superior. Tested and committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

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