Closed (fixed)
Project:
Glossary
Version:
5.x-2.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
1 Apr 2008 at 01:32 UTC
Updated:
16 Apr 2008 at 18:23 UTC
function glossary_overview($vocab, $letter = null) {
$dest = drupal_get_destination();
$tbl = get_html_translation_table(HTML_SPECIALCHARS);
$tbl[' '] = '_';
$output = '<div id="'. strtr(drupal_strtolower($vocab->name), $tbl) .'">';
if ($vocab->description) {
$output .= '<p class="glossary-description">'. $vocab->description .'</p>';
}
else {
$output = null;
}
$output is being set. If there is a description, it is being added to, else it is nulled. At the end of this function, there is:
return $output .'</div>';
}
If there is no $vocab->description, there was no <div> set to close. I removed the else { } clause from this if { } and all is well for me.
Comments
Comment #1
nancydruThat was what I was going to suggest. At one time that was needed, but it isn't any longer. I will delete that clause in my next commit. Thanks.
Comment #2
nancydruFix committed.
Comment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.