In strict reporting environments PHP complains that "Notice: Undefined variable: block_classes in include() (line 4 of .../sites/all/themes/danland/block.tpl.php)."

This patch fixes that by checking whether $block_classes exists before printing it.

CommentFileSizeAuthor
#2 1935722.patch1.85 KBBevan
#3 1935722.patch403 bytesBevan
DANLAND.patch1.02 KBBevan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bevan’s picture

Bevan CreditAttribution: Bevan commented

It also makes line-endings unix-style, which is the standard in Drupal.

Bevan’s picture

Bevan CreditAttribution: Bevan commented
FileSize
1935722.patch1.85 KB

This is also a problem in block-left.tpl.php and block-right.tpl.php. This patch fixes the problem from template.php with a new function danland_preprocess_block(). It also fixes line endings and removes deprecated CVS tags.

Bevan’s picture

Bevan CreditAttribution: Bevan commented
FileSize
1935722.patch403 bytes

This patch excludes the line ending changes and the removal of the deprecated CVS tags, so more clearly demonstrates the change for reviewing purposes.

  • jcnventura committed b355d4c on 6.x-2.x authored by Bevan 
    Issue #1935722 by Bevan: Undefined variable: block_classes
jcnventura’s picture

jcnventura CreditAttribution: jcnventura commented
Issue summary: View changes
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.