Problem/Motivation

The main page (i.e. /biblio) was returning a broken page when no nodes existed and a white page (completely broken) once I added biblio nodes.
It turns out that adding a CSS class to the main menu in my theme via
$attributes['class'][] = 'equal-height-container';
was breaking because $attributes['class'] was not an array.

Proposed resolution

I found that changing
return $output . theme('links', array('links' => $links, 'attributes' => array('class' => "biblio-export-buttons")));/
to
return $output . theme('links', array('links' => $links, 'attributes' => array('class' => array("biblio-export-buttons"))));
in line 907 of biblio_theme.inc corrected the issue.

I have attached a patch for this from the 7.x-1.x branch. I'm new to using GIT, so hopefully that's the right branch :D.

Remaining tasks

Unit tests need to be run on the patch.

CommentFileSizeAuthor
class_name.patch624 bytesmadprgmr

Comments

rjerome’s picture

Status: Needs review » Fixed

Thanks, I've committed the patch. And you get A+ for the completeness of your bug report. It probably took longer to write it than it did to fix the bug :-)

http://drupalcode.org/project/biblio.git/commit/ada2852

Ron.

Status: Fixed » Closed (fixed)

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