[drupal-devel] [bug] Fix phptemplate regressions

[adrian]

Issue status update for http://drupal.org/node/21855

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     bug reports
 Priority:     critical
 Assigned to:  TDobes
 Reported by:  adrian
 Updated by:   adrian
 Status:       patch

re: point #1 , that is a better solution.


It's all going to be removed in the next week to be replaced with a
menu system approach, but cool. thnx.




adrian



Previous comments:
------------------------------------------------------------------------

May 2, 2005 - 23: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 - 16: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 - 20:20 : Dries

Committed.  Thanks Adrian.




------------------------------------------------------------------------

May 4, 2005 - 21:25 : TDobes

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.