[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