fix duplicate function declaration and made new function default to o…

…ld behavior
hubzero · May 8, 2024 · 386f22f · 386f22f
1 parent 86cb2cb
commit 386f22f
Showing 1 changed file with 17 additions and 32 deletions.
diff --git a/core/components/com_resources/helpers/tags.php b/core/components/com_resources/helpers/tags.php
@@ -413,46 +413,31 @@ public function getTagUsage($tag, $rtrn='id')
 		return $this->_db->loadColumn();
 	}
 
-		/**
-		 * Get a tag cloud for an object
-		 * 
-		 * @param   integer  $limit
-		 * @param   string   $tagstring
-		 * @param   integer  $maxTagCount -- max count of tags that can be searched
-		 * @return  string
-		 */
-		public function getTopTagCloud($limit, $tagstring='', $maxTagCount=5)
-		{
-			$tagArray = explode(",", $tagstring);
-
-			// check that we haven't been handed too many tags to search:
-			if (count($tagArray) > $maxTagCount)
-			{
-				// nope, too many, don't search:
-				$retval = null;
-			}
-			else
-			// ok, search for the tags:
-			{
-				$tags = $this->getTopTags($limit);
-				$retval = $this->buildTopCloud($tags, 'alpha', 0, $tagstring);
-			}
-			return $retval;
-		}
-
-
 	/**
 	 * Get a tag cloud for an object
-	 *
+	 * 
 	 * @param   integer  $limit
 	 * @param   string   $tagstring
+	 * @param   integer  $maxTagCount -- max count of tags that can be searched
 	 * @return  string
 	 */
-	public function getTopTagCloud($limit, $tagstring='')
+	public function getTopTagCloud($limit, $tagstring='', $maxTagCount=null)
 	{
-		$tags = $this->getTopTags($limit);
+		$tagArray = explode(",", $tagstring);
 
-		return $this->buildTopCloud($tags, 'alpha', 0, $tagstring);
+		// check that we haven't been handed too many tags to search:
+		if ( ($maxTagCount != null) && (count($tagArray) > $maxTagCount))
+		{
+			// nope, too many, don't search:
+			$retval = null;
+		}
+		else
+		// ok, search for the tags:
+		{
+			$tags = $this->getTopTags($limit);
+			$retval = $this->buildTopCloud($tags, 'alpha', 0, $tagstring);
+		}
+		return $retval;
 	}
 
 	/**