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 3a89e52..df40ee7 100644 (file)
--- 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();