From 645da9ad8bbd4c7e4672cea26911f1705fa033df Mon Sep 17 00:00:00 2001
From: jjordaan <jarrett@knowledgetree.com>
Date: Mon, 20 Apr 2009 16:50:44 +0200
Subject: [PATCH] KTS:4183 Cloud Tag filter error handled with empty search results, instead of DB error

---
 plugins/tagcloud/TagCloudUtil.inc.php | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/plugins/tagcloud/TagCloudUtil.inc.php b/plugins/tagcloud/TagCloudUtil.inc.php
index 25f9057..cc4017b 100644
--- a/plugins/tagcloud/TagCloudUtil.inc.php
+++ b/plugins/tagcloud/TagCloudUtil.inc.php
@@ -120,7 +120,10 @@ function get_relevant_tags($oUser, $sTag)
 
         $aDocIds = DBUtil::getResultArrayKey($aQuery, 'document_id');
         $sDocs = implode(',', $aDocIds);
-
+        // Make sure user not opening a new window on tag cloud filters
+        if(!$sDocs) {
+            return array();
+        }
         // Don't display tags that have already been selected.
         $tagTree[] = $sTag;
         $cnt = count($tagTree);
@@ -135,7 +138,11 @@ function get_relevant_tags($oUser, $sTag)
         WHERE DT.document_id in ($sDocs) AND TW.tag NOT IN ($sIgnoreTags)
         GROUP BY TW.tag";
 
-        $tags = DBUtil::getResultArray(array($sQuery, $tagTree), null);
+        $tags = DBUtil::getResultArray(array($sQuery, $tagTree));
+
+        if(PEAR::isError($tags)){
+            echo $tags->getMessage();
+        }
 
         // Add new tag to the session
         if($sPrevTag != $sTag){
@@ -144,13 +151,14 @@ function get_relevant_tags($oUser, $sTag)
         }
     }
 
-    // Create tag / frequency array
     $aTags = array();
-    foreach($tags as $tag)
-    {
-        $word=$tag['tag'];
-        $freq=$tag['freq'];
-        $aTags[$word] = $freq;
+    if($tags) {
+        foreach($tags as $tag)
+        {
+            $word=$tag['tag'];
+            $freq=$tag['freq'];
+            $aTags[$word] = $freq;
+        }
     }
 
     return $aTags;
--
libgit2 0.21.4