This is very minor since most sites have side bars. There's a $layout variable initially set from phptemplate.engine inside phptemplate_page().

It's set for indicating the layout based on the presence of side block regions. Garland and a handful of other themes use this for setting classes and changing layouts.

If no sidebars are present, the variable will not be set.

CommentFileSizeAuthor
#3 phptemplate_7.patch992 bytesdrumm
#2 layout_var1.patch721 bytesdvessel
layout_var.patch508 bytesdvessel

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
Version: 5.1 » 6.x-dev
Status: Reviewed & tested by the community » Needs work

Instead, we could initialize the variable to 'none' above

if ($show_blocks) {

and avoid having an if statement.

dvessel’s picture

dvessel commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new layout_var1.patch721 bytes

Done.

drumm’s picture

drumm
he/him
Location NY, US
commented
StatusFileSize
new phptemplate_7.patch992 bytes

Rerolled from Drupal's root directory and changing nearby comment style.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

drumm’s picture

drumm
he/him
Location NY, US
commented

Committed to 5

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)