[drupal-devel] [bug] $site redundant in theme('page')
Dries
drupal-devel at drupal.org
Sat Jul 30 14:05:36 UTC 2005
Issue status update for
http://drupal.org/node/27791
Post a follow up:
http://drupal.org/project/comments/add/27791
Project: Drupal
Version: cvs
Component: theme system
Category: bug reports
Priority: normal
Assigned to: robertDouglass
Reported by: robertDouglass
Updated by: Dries
Status: patch
I think it is being used in bluemarine/page.tpl.php:
l(t('edit secondary links'),'admin/themes/settings/' .$theme)
Please double check.
Dries
Previous comments:
------------------------------------------------------------------------
Fri, 29 Jul 2005 17:41:48 +0000 : robertDouglass
Attachment: http://drupal.org/files/issues/phptemplate_redundant_var_patch.txt (815 bytes)
The $site variable is made redundant by $site_name, and is not used.
------------------------------------------------------------------------
Fri, 29 Jul 2005 21:24:19 +0000 : Steven
Bluemarine declares $theme as global, but as far as I can tell it
doesn't use it anywhere. I think we can safely remove it?
------------------------------------------------------------------------
Sat, 30 Jul 2005 10:07:33 +0000 : robertDouglass
agree. It can be removed.
More information about the drupal-devel
mailing list