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

Comments

Bevan’s picture

Bevan commented

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

Bevan’s picture

Bevan commented
StatusFileSize
new 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 commented
StatusFileSize
new 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
Primary language English
commented
Issue summary: View changes
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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