From 0688a8b97b7d310edfd92a70519100838f584b51 Mon Sep 17 00:00:00 2001 From: "Michael J. Rubinsky" Date: Mon, 8 Mar 2010 18:10:56 -0500 Subject: [PATCH] Call getTagIds() instead of ensureTags() so searching for a tag doesn't alwasy result in it being added to the rampage_tags table. --- kronolith/lib/Tagger.php | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) 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(); -- 2.11.0