Issue status update for http://drupal.org/node/21855 Project: Drupal -Version: 4.6.0 +Version: cvs Component: theme system -Category: tasks +Category: bug reports -Priority: normal +Priority: critical -Assigned to: adrian +Assigned to: TDobes Reported by: adrian Updated by: TDobes -Status: fixed +Status: patch Attachment: http://drupal.org/files/issues/fix-phptemplate-regressions.patch (9.06 KB) The recent commit of phptemplate caused a number of problems for non-phptemplate themes. A patch is attached to address these issues. Changes include: * parsing of the primary/secondary links has been moved out of phptemplate and into theme_get_setting. * unnecessary and XHTML-invalidating duplicate div#help removed from themes/bluemarine/page.tpl.php (this is already generated by theme_help) * weird generation of the "edit primary/secondary links" messages removed from bluemarine and placed in theme.inc * unnecessary changes to themes/bluemarine/style.css rolled back (the phptemplate bluemarine had an older version of style.css than the one in core) * chameleon updated to work with new link scheme (passes links through theme_links) Please review and/or apply. I recommend that the themes/bluemarine/xtemplate.xtmpl file be deleted in the next CVS commit as well. I assume it's an accidental leftover. Also, please look into the parse error in database/updates.inc. It looks like another patch might have been mixed up accidentially. After this patch lands, I'll update all contrib themes and the theme upgrade guide. TDobes Previous comments: ------------------------------------------------------------------------ May 2, 2005 - 15:22 : adrian Attachment: http://drupal.org/files/issues/phptemplate4core.tar.bz2 (21.32 KB) PHPTemplate has been destined to move to core for a while, here is the patch and the updated version of phptemplate. As discussed on the mailing list, this includes the phptemplate port of bluemarine, and moves the link settings to admin/themes/settings. ------------------------------------------------------------------------ May 4, 2005 - 08:03 : adrian Attachment: http://drupal.org/files/issues/phptemplate4core.tar_0.bz2 (19.78 KB) Here is an updated phptemplate that : 1) uses node_load to retrieve the node, if on a node page. 2) changes the seqid variable to id. 3) removes the readme.txt file 4) does away with the template.php file, and moves the functionality into phptemplate.engine ------------------------------------------------------------------------ May 4, 2005 - 12:20 : Dries Committed. Thanks Adrian.