I just want to use this module for the template override functionality. I have tried setting the logo path to blank but it still outputs the <img> tag.

Here is a fix for it (in logotool_preprocess_page());

<?php
    case LOGOTOOL_NORMAL:
      $logo_path = variable_get('logotool_path', 'misc/druplicon.png');
      if (!empty($logo_path)) {
        $variables['logo'] = $base_path . $logo_path;
      }
      break;
?>

Comments

pobster’s picture

pobster commented
Status: Active » Closed (won't fix)

A better idea would be to split this out into a separate sub-module I suppose, but I'm not sure why you would install this module to (only) achieve this and not just do it in your theme layer? I can't commit this, the idea is nice but this complicated functionality is only really for themers/ devs and they should know already how to use preprocess hooks!

Thanks,

Pobster

pobster’s picture

pobster commented
Status: Closed (won't fix) » Patch (to be ported)

Hmmm I've changed my mind, I'll commit this... The module shouldn't always assume the logo path will be present - my comments about this functionality not needing a UI are still valid though!

Thanks,

Pobster

mstrelan’s picture

mstrelan commented

I agree with you that the module shouldn't always assume the logo path is present. I also agree with you that this should be done on the theme layer, but my client wants to change this and is comfortable using your module but not comfortable touching PHP.

pobster’s picture

pobster commented
Issue summary: View changes
Status: Patch (to be ported) » Closed (outdated)

Closing as D6.x is now unsupported.