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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm 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 CreditAttribution: dvessel commented
Status: Needs work » Reviewed & tested by the community
FileSize
layout_var1.patch721 bytes

Done.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
FileSize
phptemplate_7.patch992 bytes

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

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented

Committed to 5

Anonymous’s picture

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