CommentFileSizeAuthor
#7 zen.patch8.54 KBjohnalbin

Comments

johnalbin’s picture

Category: task » feature

subscribing. :-)

michelle’s picture

Category: feature » task

Just fixing the category so it doesn't get forgotten.

Michelle

Keith Hurst’s picture

Subscribing also, as I'm having no end of problems with Zen and ADVForum. Will examine the multiple threads on this later.

Thanks Michelle, it's a fantastic job you're doing.

Keith

michelle’s picture

Status: Active » Postponed (maintainer needs more info)

path_to_subtheme has been added but is untested. I'd appreciate it if someone who uses Zen could give it a try.

I'm drawing a blank on what I'm supposed to do with the new Zen preprocess functions... Could you refresh my memory, please?

Thanks,

Michelle

annez’s picture

It's not working for me, though I may have something set up wrong. It looks like advanced_forum_theme_path in advanced_forum.module doesn't see path_to_subtheme even though I can call it from my template.php. Am investigating but maybe someone with more knowledge of Drupal and/or PHP will have an idea about what's going wrong. Do functions defined in templates get added later maybe?

michelle’s picture

Status: Postponed (maintainer needs more info) » Active

Darn, I was hoping it would just work... I guess I'll need to install Zen and figure it out. Thanks for the feedback.

Michelle

johnalbin’s picture

Status: Active » Needs review
StatusFileSize
new8.54 KB

Okay, this patch does 3 things…

  1. Changes theme_image(...) to theme('image', ...). (Not actually related to this bug, but still a theme-layer bug.)
  2. Changes _phptemplate_callback() to use a proper $suggestions array. For example,
    _phptemplate_callback("$forum_theme/advf-forum-topic-list", $variables) becomes
    _phptemplate_callback("advf_forum_topic_list", $variables, array("$forum_theme/advf-forum-topic-list"))
  3. Adds theme_init() before checking for the existance of path_to_subtheme, since advforum seems to be checking for its existence before drupal theme layer is started (i.e. before Zen's template.php is read.)

And that should fix the final issues with Zen and advforum. :-)

michelle’s picture

Status: Needs review » Fixed

Comitted. Thanks!

Michelle

excell’s picture

excuse me - could you tell me please what file I put the patch into?

Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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