Closed (won't fix)
Project:
Drupal core
Version:
5.x-dev
Component:
theme system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
13 Sep 2008 at 16:02 UTC
Updated:
22 Jun 2011 at 00:18 UTC
Jump to comment: Most recent file
Comments
Comment #1
Czymra commentedNot sure at all if this is related but my custom regions seem to conflict with the panels 'header' region. I however use the Panel Module. Sorry if this is is unrelated.
I'm on 5.10 as well.
Error messages:
warning: Cannot modify header information - headers already sent by (output started at /usr/local/psa/home/vhosts/url/httpdocs/drupal5/sites/all/themes/symposium/template.php:3) in /usr/local/psa/home/vhosts/url/httpdocs/drupal5/includes/common.inc on line 141.
warning: Cannot modify header information - headers already sent by (output started at /usr/local/psa/home/vhostsurl/httpdocs/drupal5/sites/all/themes/symposium/template.php:3) in /usr/local/psa/home/vhosts/url/httpdocs/drupal5/includes/common.inc on line 141.
Comment #2
owen barton commentedTracked this down to http://drupal.org/node/281042
Probably will patch there, but I am leaving this open for others (or in case that issue is somehow the correct behavior).
Comment #3
owen barton commentedI tracked this down to the following code (broken out for readability) in phptemplate.engine:
There doesn't seem to be a simple way of reverting this code, without also reverting the original issue (which is that blocks were being generated on 404 pages, a significant performance issue).
In addition, it could be argued that this is the expected behavior from panels point of view, because the description on the checkbox is "'Check this to have the panel page disable all regions displayed in the theme.'", which suggests that header and other regions will indeed be empty of configured blocks. However this is a change of behavior, since in Drupal 5.8 (and prior) the blocks were always displayed in these regions.
Either this needs to be fixed in core (which seems a little unlikely now, perhaps - unless other problems are found), or needs to be fixed in panels.
This could be done simply by allowing the page theme to load the blocks (see attached patch). This leads to a bit of redundancy because the sidebars are generated but never shown (AFAICS). There is probably a more elegant solution out there though.
Ideally it might be nice to make this panels control a radio to allow you to leave blocks as is, disable just the left/right region blocks, or disable *all* region blocks.
Comment #4
sdboyer commentedslating for my work time this weekend
Comment #5
drummThis patch doesn't make sense, there is no second argument to theme('page') in Drupal 5, so the FALSE goes nowhere.
Since this is a behavior change, then it can be fixed in Drupal 5.x. This issue should be used so we can keep this specific issue separated out.
Comment #6
owen barton commentedI am not quite sure where this was fixed, or how - but as of 5.14 and the latest panels 2 I can no longer reproduce this. Is anyone else still able to?
Comment #7
dpearcefl commentedConsidering the time elapsed between now and the last comment plus the fact that D5 is no longer supported, I am closing this ticket.