no_fly_list 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da62f66e9694575999da6c6864650b05036fe0f32ed762255d4552e11fc53ce4
-  data.tar.gz: 965687001501fec670c74c1a89d95340776b3b8ffa061517d34c430439471572
+  metadata.gz: 8e5fd47ff5074ba83503a9c1633bada42f7348aafb9a914d2c5f51a70820427d
+  data.tar.gz: a337a25cdce43d598fd501266ba0d99665c3adb183f994121fd201082e4e874b
 SHA512:
-  metadata.gz: 3c347ef635cb9c1adeb14df10b7ce8a6b6220104f93b0fc4b7debccf30c0537da86e232a4b62000fb3498253c8ae5535c5ca9ddb9290371d3afd4dd021c894ab
-  data.tar.gz: 3b1685788f672eadfe558f2bbceacdfd287dbcb865b7e2bbd8b718e710cf3870a575900bcae69452ebc9a720f8ca77c4c0f2b4488d56431e6c5a30946f4d630b
+  metadata.gz: a0063a48bd720527540437b156136c704bd9deea0c4985fb34a9c4a7595c3c932d3c9b13ab5b8032daf277b5e547b358b5343410bdb6bedae1e466bc081edecd
+  data.tar.gz: 2f430e49bc7a6cc8ed09b1aa5686121c8d90d2262f7a6cce48de48e208804ab77899227e2b48ace8a6b017a9147fd368136ce42b99eeaab9668361e1bf10acfc

data/lib/no_fly_list/taggable_record/config.rb

@@ -24,7 +24,8 @@ module NoFlyList
         restrict_to_existing: options.fetch(:restrict_to_existing, false),
         limit: options.fetch(:limit, nil),
         case_sensitive: options.fetch(:case_sensitive, true),
-        adapter: @adapter
+        adapter: @adapter,
+        counter_cache: options.fetch(:counter_cache, false)
       }
     end
 

data/lib/no_fly_list/taggable_record/configuration.rb

@@ -94,7 +94,8 @@ module NoFlyList
 
           belongs_to :taggable,
                      class_name: setup.taggable_klass.name,
-                     foreign_key: "taggable_id"
+                     foreign_key: "taggable_id",
+                     counter_cache: setup.counter_cache ? "#{setup.context}_count" : nil
 
           include NoFlyList::TaggingRecord
         end
@@ -156,12 +157,14 @@ module NoFlyList
 
       # Sets up associations for local (non-global) tags
       def setup_local_tag_associations(setup, singular_name)
+        plural_name = setup.context.to_s
         # Set up tag class associations
         setup.tag_class_name.constantize.class_eval do
           has_many :"#{singular_name}_taggings",
                    -> { where(context: singular_name) },
                    class_name: setup.tagging_class_name,
                    foreign_key: "tag_id",
+                   counter_cache: setup.counter_cache ? "#{plural_name}_count" : nil,
                    dependent: :destroy
 
           has_many :"#{singular_name}_taggables",

data/lib/no_fly_list/taggable_record/tag_setup.rb

@@ -4,7 +4,7 @@ module NoFlyList
   module TaggableRecord
     class TagSetup
       attr_reader :taggable_klass, :context, :transformer, :polymorphic,
-                  :restrict_to_existing, :limit,
+                  :restrict_to_existing, :limit, :counter_cache,
                   :tag_class_name, :tagging_class_name, :adapter
 
       def initialize(taggable_klass, context, options = {})
@@ -13,6 +13,8 @@ module NoFlyList
         @transformer = options.fetch(:transformer, ApplicationTagTransformer)
         @polymorphic = options.fetch(:polymorphic, false)
         @restrict_to_existing = options.fetch(:restrict_to_existing, false)
+        @counter_cache = options.fetch(:counter_cache, false)
+        @counter_cache_column = "#{context}_count"
         @limit = options.fetch(:limit, nil)
         @tag_class_name = determine_tag_class_name(taggable_klass, options)
         @tagging_class_name = determine_tagging_class_name(taggable_klass, options)

data/lib/no_fly_list/tagging_proxy.rb

@@ -74,8 +74,11 @@ module NoFlyList
           end
 
           # Clear existing tags
+          old_count = model.send(context_taggings).count
           model.send(context_taggings).delete_all
 
+          # Update counter
+          model.update_column("#{@context}_count", 0) if setup[:counter_cache]
           # Create new tags
           @pending_changes.each do |tag_name|
             tag = find_or_create_tag(tag_name)
@@ -95,6 +98,8 @@ module NoFlyList
             model.send(context_taggings).create!(attributes)
           end
         end
+        # Update counter to match the actual count
+        model.update_column("#{@context}_count", @pending_changes.size) if setup[:counter_cache]
 
         refresh_from_database
         true
@@ -147,10 +152,10 @@ module NoFlyList
       "#<#{self.class.name} tags=#{current_list.inspect} transformer_with=#{transformer_name} >"
     end
 
-    def add(tag)
+    def add(*tags)
       return self if limit_reached?
 
-      new_tags = transformer.parse_tags(tag)
+      new_tags = tags.flatten.map { |tag| transformer.parse_tags(tag) }.flatten
       return self if new_tags.empty?
 
       @pending_changes = current_list + new_tags
@@ -158,8 +163,8 @@ module NoFlyList
       self
     end
 
-    def add!(tag)
-      add(tag)
+    def add!(*tags)
+      add(*tags)
       save
     end
 
@@ -179,12 +184,14 @@ module NoFlyList
       old_list = current_list.dup
       @pending_changes = []
       mark_record_dirty if @pending_changes != old_list
+      model.write_attribute("#{@context}_count", 0) if setup[:counter_cache]
       self
     end
 
     def clear!
       @model.send(@context.to_s).destroy_all
       @pending_changes = []
+      @model.update_column("#{@context}_count", 0) if setup[:counter_cache]
       self
     end
 

data/lib/no_fly_list/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module NoFlyList
-  VERSION = "0.3.0"
+  VERSION = "0.4.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: no_fly_list
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Abdelkader Boudih