Empty Vocab breaks the HTML

nicholasThompson - April 3, 2008 - 16:09
Project:SiteMenu
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

This module is great, but if you have an empty vocabulary it can break the page horribly - it ends up returning a without opening one. This can completely change the structure of the HTML on the page (as it did on one of my sites).

The fix is simple - at the head of the theme function simply return an emty string if the $tree array is empty.

AttachmentSize
sitemenu_empty_vocab_problem.patch616 bytes

#1

kbahey - April 3, 2008 - 16:13
Status:needs review» fixed

Thank you.

Committed to 5.x-1.x-dev.

#2

Anonymous (not verified) - April 17, 2008 - 16:21
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.