Blocks are hidden in 404 pages

This is not a solution to the present problem, but something to be considered by the module designer for the next version of the module.

1) Throttle: What is you require throttle to be turned on by default for search-404, and throttle off all the blocks. If the owner of a given website has enough resources to handle the performance demand, then he/she can turn-on the blocks

2) Caching: You can set search-404 to require block caching or to only display cached blocks so that if the site admin wants blocks to show, then he/she must set those particular blocks to be cached. This will solve the theming issue without introducing the performance issue

-----
iDonny Productions: Drupal CMS Implementation, Theme UI/UX Design & Development, and Web Standards

but not interfere with the hiding of blocks on other parts of my site.

You may still use the Admin \ Blocks interface to specify certain URL's to hide certain blocks. This trick does not override that.

I did not use your handy helper for themes section, just the bit that controls the 404 page.

So my code is as follows:

/**
 * Override or insert PHPTemplate variables into the templates.
 */
function phptemplate_preprocess_page(&$vars) {
  $vars['tabs2'] = menu_secondary_local_tasks();
  
  // Hook into color.module
  if (module_exists('color')) {
    _color_page_alter($vars);
  }

// had to remove a closing brace here, but technically it should be there?

// Only does the check if required - see this node for explanation http://drupal.org/node/129762
  if(!$vars['show_blocks']) {
    global $theme;
    $regions = system_region_list($theme);
    foreach (array_keys($regions) as $region) {
      // Only set left and right regions
      // Drupal core sets the other blocks already
      // IMHO this shows a real lack of design considerations for leaving these out!
      if ($region == 'left' || $region == 'right') {
        $blocks = theme('blocks', $region);
        if(isset($variables[$region])) {
          $vars[$region] .= $blocks;
        }
        else {
          $vars[$region] = $blocks;
        }
      }
    }
  }
} 

Once I removed the closing bracket (which php error notice emitted) the template with the fix was working. I don't quite understand why, I changed nothing else, and that extra curly brace should be there? Thanks if you can advise further.

Renaee.

Hi Alan,
This works to add the sidebars back in :)

But it doesn't change the body class (still has 'no-sidebars') -- so the info overlaps :(

I'm using a zen subtheme - based on the current version zen theme.

Any advice for this?

Cheers!
Scott

The block creation code done by Drupal is the first in the preprocess_page function chain. The theme is the last in this chain.

So if anything in between changes that is dependent on the regions, this will need to be mimicked here. As per the body class issue noted.

Just used the function for the first time using a theme that required the body classes, and the following modifications were needed to mimic the body classes. This is independent on the order of classes within the body_classes variable, as these could change without notice.

<?php
/**
* Override or insert PHPTemplate variables into the templates.
*/
function phptemplate_preprocess_page(&$vars) {
  // Only does the check if required
  if(!$vars['show_blocks']) {
    global $theme;
    $regions = system_region_list($theme);
    foreach (array_keys($regions) as $region) {
      if ($region == 'left' || $region == 'right') {
        $blocks = theme('blocks', $region);
        if(isset($variables[$region])) {
          $vars[$region] .= $blocks;
        }
        else {
          $vars[$region] = $blocks;
        }
      }
    }
    // redoing core classes
    // Set up layout variable.
    $vars['layout'] = 'none';
    if (!empty($vars['left'])) {
      $vars['layout'] = 'left';
    }
    if (!empty($vars['right'])) {
      $vars['layout'] = ($vars['layout'] == 'left') ? 'both' : 'right';
    }
    // redo body classes
    $body_classes = explode(' ', $vars['body_classes']);
    foreach($body_classes as $index => $bc) {
      if ($bc == 'no-sidebars') {
        unset($body_classes[$index]);
      }
    }
    // Add information about the number of sidebars.
    if ($vars['layout'] == 'both') {
      $body_classes[] = 'two-sidebars';
    }
    elseif ($vars['layout'] == 'none') {
      $body_classes[] = 'no-sidebars';
    }
    else {
      $body_classes[] = 'one-sidebar sidebar-'. $vars['layout'];
    }
    // Implode with spaces.
    $vars['body_classes'] = implode(' ', $body_classes);
  }
}
?>

Greetings Alan-

I just realized that domains I upgraded to D6 were not displaying all blocks on 404 pages.

For D5 I had added code to template.php as suggested above on this thread.

Was your code intended to also go in template.php to show blocks on D6 404 pages, or in some other file? I am not receiving all blocks. I also tried the other solution, a customization of your solution, and that causes a PHP error.

Thanks!

I put this code but not work.

i need to show header blocks in custom page not found

That's a good idea. I'm not aware of any other pages where the block regions would not show but sometimes it's better to play it safe.

⎋ joon park
–www.dvessel.com–

This worked nicely, thank you.
----------------------------------------------------------------------
http://classicvinyl.biz
http://music.bwv810.com
http://davidhertzberg.com
http://association.drupal.org/user/1207

I am a writer, researcher and solo drupal freelancer

Continuing this on one step further:

<?php
function YOURTHEME_page($content, $show_blocks = TRUE) {
  // Set additional conditions to change $show_blocks if you want here...
  $is404 = $_GET['q'] == drupal_get_normal_path(variable_get('site_404', ''));
  $is403 = $_GET['q'] == drupal_get_normal_path(variable_get('site_403', ''));
  $show_blocks = ($is404 || $is403) ? true : $show_blocks;
  // phptemplate_page is what gets called usually but the
  // theme function allows us to override it with the themes' name.
  // We're calling it directly here so we don't have to copy tons of code.
  return phptemplate_page($content, $show_blocks);
}
?>