[drupal-devel] [bug] $site redundant in theme('page')

Steven drupal-devel at drupal.org
Sat Jul 30 15:01:23 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:   Steven
-Status:       fixed
+Status:       patch

Sorry the two issues ($site and $theme) got mixed up. I probably just
pasted in the wrong tab :P.


$site is being used by several contrib modules... $site_name only
applies to the main name when displayed as a logo. Contrib usages show
"Welcome to $site". It's probably a good idea to keep it, no?




Steven



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.




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

Sat, 30 Jul 2005 14:05:32 +0000 : Dries

I think it is being used in bluemarine/page.tpl.php:
l(t('edit secondary links'),'admin/themes/settings/' .$theme)


Please double check.




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

Sat, 30 Jul 2005 14:53:09 +0000 : Steven

This is not present in the core bluemarine, I removed it.







More information about the drupal-devel mailing list