I am working on http://drupal.org/node/461938 and wanted to test for the slogan being properly escaped to find out that there no longer it is no longer in garlands page.tpl.php
http://api.drupal.org/api/drupal/themes--garland--page.tpl.php/7/source

The default still uses $site_slogan.
http://api.drupal.org/api/drupal/modules--system--page.tpl.php/7/source

I heard the slogan will be moved to a block? I'd say its a bug?

Comments

todd nienkerk’s picture

todd nienkerk commented

$site_slogan is still available to the default page.tpl as shown in <a href="http://api.drupal.org/api/function/template_preprocess_page/7">template_preprocess_page()</a>.

Garland implements this in a custom way in its implementation of garland_preprocess_page() in template.php:

  // Set a variable for the site name title and logo alt attributes text.
  $slogan_text = filter_xss_admin(variable_get('site_slogan', ''));
  $site_name_text = filter_xss_admin(variable_get('site_name', 'Drupal'));
  $vars['site_name_and_slogan'] = $site_name_text . ' ' . $slogan_text;

Marking this "by design," as the difference is intentional.

todd nienkerk’s picture

todd nienkerk commented
Status: Active » Closed (works as designed)