a little more protection against empty tags
This commit is contained in:
		| @@ -147,7 +147,7 @@ class TagSub extends Managed_DataObject | ||||
|         $keypart = sprintf('tagsub:by_profile:%d', $profile->id); | ||||
|         $tagstring = self::cacheGet($keypart); | ||||
|          | ||||
|         if ($tagstring !== false) { | ||||
|         if ($tagstring !== false && !empty($tagstring)) { | ||||
|             $tags = explode(',', $tagstring); | ||||
|         } else { | ||||
|             $tagsub = new TagSub(); | ||||
| @@ -155,7 +155,9 @@ class TagSub extends Managed_DataObject | ||||
|  | ||||
|             if ($tagsub->find()) { | ||||
|                 while ($tagsub->fetch()) { | ||||
|                     $tags[] = $tagsub->tag; | ||||
|                     if (!empty($tagsub->tag)) { | ||||
|                         $tags[] = $tagsub->tag; | ||||
|                     } | ||||
|                 } | ||||
|             } | ||||
|  | ||||
|   | ||||
| @@ -62,12 +62,14 @@ class TagSubMenu extends Menu | ||||
|         $this->out->elementStart('ul', array('class' => 'nav')); | ||||
|  | ||||
|         foreach ($this->tags as $tag) { | ||||
|             if (!empty($tag)) { | ||||
|                 $this->out->menuItem(common_local_url('tag', | ||||
|                                                       array('tag' => $tag)), | ||||
|                                      sprintf('#%s', $tag), | ||||
|                                      sprintf(_('Notices tagged with %s'), $tag), | ||||
|                                      $this->actionName == 'tag' && $this->action->arg('tag') == $tag, | ||||
|                                      'nav_streams_tag_'.$tag); | ||||
|             } | ||||
|         } | ||||
|  | ||||
|         $this->out->elementEnd('ul'); | ||||
|   | ||||
		Reference in New Issue
	
	Block a user