Block module adds empty theme regions to the $page array [#674406]

Moshe Weitzman noticed that in the top level of the $page array (on every page), there are empty elements for 'dashboard_main' and 'dashboard_sidebar'.

Looking into it, that has nothing to do with the dashboard module - turns out there are also empty elements added for any theme region that does not have blocks displayed on the current page.

Seems a bit wasteful, and can't think of any reason why they need to be added as placeholders or anything, so this patch just removes them.

Comment	File	Size	Author
	blocks-empty-regions.patch	730 bytes	David_Rothstein

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

moshe weitzman CreditAttribution: moshe weitzman commented 5 January 2010 at 03:43

Status:

Needs review

» Reviewed & tested by the community

We added these empty regions originally in order to avoid notices later on when we were iterating over regions in the preprocess stage. If the test bot does not find any notices, this seems reasonable ... I didn't recognize these as regions because they don't have a #theme_wrappers on them.