From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Mon, 8 Mar 2010 23:10:56 +0000 (-0500)
Subject: Call getTagIds() instead of ensureTags() so searching for a tag
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=0688a8b97b7d310edfd92a70519100838f584b51;p=horde.git

Call getTagIds() instead of ensureTags() so searching for a tag
doesn't alwasy result in it being added to the rampage_tags table.
---

diff --git a/kronolith/lib/Tagger.php b/kronolith/lib/Tagger.php
index 3a89e5252..df40ee76f 100644
--- a/kronolith/lib/Tagger.php
+++ b/kronolith/lib/Tagger.php
@@ -157,11 +157,6 @@ class Kronolith_Tagger
     /**
      * Search for resources that are tagged with all of the requested tags.
      *
-     * TODO: Change this to use something like a Content_Tagger::tagExists() or
-     *       possibly add a $create = true parameter to ensureTags()
-     *       so searching for any arbitrary text string won't cause the string
-     *       to be added to the rampage_tags table as a tag (via ensureTags)
-     *
      * @param array $tags  Either a tag_id, tag_name or an array.
      * @param array $filter  Array of filter parameters.
      *   (string)typeId      - only return either events or calendars, not both.
@@ -211,7 +206,7 @@ class Kronolith_Tagger
         }
 
         /* Add the tags to the search */
-        $args['tagId'] = $this->_tagger->ensureTags($tags);
+        $args['tagId'] = $this->_tagger->getTagIds($tags);
 
         /* Restrict to events or calendars? */
         $cal_results = $event_results = array();