This code in theme_get_setting() assumes the "default" logo.png file exists in the theme and causes a 404 when it does not:

// Generate the path to the logo image.
if ($cache[$theme]['toggle_logo']) {
  if ($cache[$theme]['default_logo']) {
    $cache[$theme]['logo'] = file_create_url(dirname($theme_object->filename) . '/logo.png');
  }
  elseif ($cache[$theme]['logo_path']) {
    $cache[$theme]['logo'] = file_create_url($cache[$theme]['logo_path']);
  }
}

Shouldn't the default logo be misc/druplicon.png? If not, shouldn't we check if the file exists in template_preprocess_page() before populating the $logo variable?

Comments

Jeff Burnz’s picture

Jeff Burnz commented

If we don't fix this it has to be documented - that themes must have a logo.

bleen’s picture

bleen commented

if we're going to check that the file exists (which I think we should) shouldn't we check it right there in them_get_settings so that the result is cached .. not in template_preprocess_page?

Thoughts?

jacine’s picture

jacine
she/her
Location New York
commented

I just don't get where the default Drupalicon logo went, but yeah theme_get_settings() is better.

Status: Active » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.