mongoid_taggable 0.1.6 → 0.1.7

data/Rakefile

@@ -7,7 +7,7 @@ begin
   require 'jeweler'
   Jeweler::Tasks.new do |gemspec|
     gemspec.name = "mongoid_taggable"
-    gemspec.version = "0.1.6"
+    gemspec.version = "0.1.7"
     gemspec.summary = "Mongoid taggable behaviour"
     gemspec.description = "Mongoid Taggable provides some helpers to create taggable documents."
     gemspec.email = "wilkerlucio@gmail.com"

data/lib/mongoid/taggable.rb

@@ -15,17 +15,21 @@
 module Mongoid::Taggable
   def self.included(base)
     # create fields for tags and index it
-    base.field :tags_array, :type => Array
+    base.field :tags_array, :type => Array, :default => []
     base.index [['tags_array', Mongo::ASCENDING]]
 
     # add callback to save tags index
     base.after_save do |document|
-      document.class.save_tags_index!
+      if document.tags_array_changed
+        document.class.save_tags_index!
+        document.tags_array_changed = false
+      end
     end
 
     # extend model
     base.extend         ClassMethods
     base.send :include, InstanceMethods
+    base.send :attr_accessor, :tags_array_changed
 
     # enable indexing as default
     base.enable_tags_index!
@@ -41,21 +45,19 @@ module Mongoid::Taggable
     def tagged_with_all(*tags)
       self.all_in(:tags_array => tags.flatten)
     end
-    
+
     def tagged_with_any(*tags)
       self.any_in(:tags_array => tags.flatten)
     end
 
     def tags
-      db = Mongoid::Config.master
-      db.collection(tags_index_collection).find.to_a.map{ |r| r["_id"] }
+      tags_index_collection.master.find.to_a.map{ |r| r["_id"] }
     end
 
     # retrieve the list of tags with weight (i.e. count), this is useful for
     # creating tag clouds
     def tags_with_weight
-      db = Mongoid::Config.master
-      db.collection(tags_index_collection).find.to_a.map{ |r| [r["_id"], r["value"]] }
+      tags_index_collection.master.find.to_a.map{ |r| [r["_id"], r["value"]] }
     end
 
     def disable_tags_index!
@@ -71,16 +73,17 @@ module Mongoid::Taggable
       @tags_separator || ','
     end
 
-    def tags_index_collection
+    def tags_index_collection_name
       "#{collection_name}_tags_index"
     end
 
+    def tags_index_collection
+      @@tags_index_collection ||= Mongoid::Collection.new(self, tags_index_collection_name)
+    end
+
     def save_tags_index!
       return unless @do_tags_index
 
-      db = Mongoid::Config.master
-      coll = db.collection(collection_name)
-
       map = "function() {
         if (!this.tags_array) {
           return;
@@ -101,7 +104,7 @@ module Mongoid::Taggable
         return count;
       }"
 
-      coll.map_reduce(map, reduce, :out => tags_index_collection)
+     self.collection.master.map_reduce(map, reduce, :out => tags_index_collection_name)
     end
   end
 
@@ -112,6 +115,7 @@ module Mongoid::Taggable
 
     def tags=(tags)
       self.tags_array = tags.split(self.class.tags_separator).map(&:strip).reject(&:blank?)
+      @tags_array_changed = true
     end
   end
 end

data/mongoid_taggable.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{mongoid_taggable}
-  s.version = "0.1.6"
+  s.version = "0.1.7"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Wilker Lucio", "Kris Kowalik"]
-  s.date = %q{2011-05-04}
+  s.date = %q{2011-07-01}
   s.description = %q{Mongoid Taggable provides some helpers to create taggable documents.}
   s.email = %q{wilkerlucio@gmail.com}
   s.extra_rdoc_files = [

data/spec/mongoid/taggable_spec.rb

@@ -17,10 +17,17 @@ require File.join(File.dirname(__FILE__), %w[.. spec_helper])
 class MyModel
   include Mongoid::Document
   include Mongoid::Taggable
+  field :name
 end
 
 describe Mongoid::Taggable do
 
+  describe "default tags array value" do
+    it 'should be an empty array' do
+      MyModel.new.tags_array.should == []
+    end
+  end
+
   context "finding" do
     let(:model){MyModel.create!(:tags => "interesting,stuff,good,bad")}
     context "by tagged_with" do
@@ -119,7 +126,15 @@ describe Mongoid::Taggable do
 
   context "indexing tags" do
     it "should generate the index collection name based on model" do
-      MyModel.tags_index_collection.should == "my_models_tags_index"
+      MyModel.tags_index_collection_name.should == "my_models_tags_index"
+    end
+
+    it "should generate the index collection model based on model" do
+      MyModel.tags_index_collection.should be_a Mongoid::Collection
+    end
+
+    it "should generate the index collection model based on model with the collection name" do
+      MyModel.tags_index_collection.name.should == "my_models_tags_index"
     end
 
     context "retrieving index" do
@@ -160,5 +175,22 @@ describe Mongoid::Taggable do
         MyModel.tags.should == []
       end
     end
+
+    it 'should launch the map/reduce if index activate and tag_arrays change' do
+      m = MyModel.create!(:tags => "food,ant,bee")
+      m.tags = 'juice,food'
+      MyModel.collection.master.should_receive(:map_reduce)
+      m.save
+    end
+
+    it 'should not launch the map/reduce if index activate and tag_arrays not change' do
+      m = MyModel.create!(:tags => "food,ant,bee")
+      MyModel.collection.master.should_not_receive(:map_reduce)
+      m.save
+      m.name = 'hello'
+      m.save
+    end
+
   end
-end
+
+end

metadata

@@ -2,7 +2,7 @@
 name: mongoid_taggable
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors: 
 - Wilker Lucio
@@ -11,7 +11,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-04 00:00:00 Z
+date: 2011-07-01 00:00:00 Z
 dependencies: []
 
 description: Mongoid Taggable provides some helpers to create taggable documents.