The change I made to get the buggers to go away:

/**
 * Override or insert PHPTemplate variables into the templates.
 */
function phptemplate_preprocess_page(&$vars) {
  // Hook into color.module
  if (module_exists('color')) {
    _color_page_alter($vars);
  }
  // Avoid E_ALL notices
  if (empty($vars['node'])) {
    $vars['node'] = FALSE;
  }
  if (empty($vars['content_top'])) {
    $vars['content_top'] = FALSE;
  }
}

Comments

pixture’s picture

pixture commented
Status: Needs review » Fixed

Hi, thank you for the bug report and code patch.

Although your code will solve E_ALL notice issue, I want to fix them fundamentally so I modified page.tpl.php.

$content-top was the left over of my other themes and it is not necessary to be there with these themes.
So I got rid of it from page.tpl.php. I also changed if($node) to if(!empty($node)) to avoid E_ALL notice in page.tpl.php.

I have just released this fix for all three themes. (revision 6.x-1.1)
Again, thank you for the bug report and your code.

robertdouglass’s picture

robertdouglass commented

Woohoo! The very model of a modern major theme maintainer =D

pixture’s picture

pixture commented
Version: 6.x-1.0 » 6.x-1.1
Assigned: Unassigned » pixture
Status: Fixed » Closed (fixed)

Since the bug was fixed, I think I should close this thread.