no_fly_list 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f1516f63b4ece62151501ea60f15ae1d5c2c3ea5738f22881ad5e2456a74b68
-  data.tar.gz: c9b88ce48a93a03fd21ade6eccfb0210e788c26ea78150ce92ee5ecf5f0ce8b0
+  metadata.gz: a83b9c51a6624a462debf10d93389d093ab732e9b7e836ee90bde0f1ae5239e5
+  data.tar.gz: 88e96def76873eaa415431ce3ddb05076f59234d68583bad4d8c4f298fe9c0c4
 SHA512:
-  metadata.gz: ea013a28e4c1246f511ae736951a89a2c292668a6d3ce1ecbf2ada940cee52576144f5de4cfebd2f10a38feb3a80f94082cc1d6ab340d692fe14f75e329b0913
-  data.tar.gz: 9b714d9919bcf14c9fda34bada054f49af0202ab1189f0e487f38d1140df4f27e5580ef3b737adf0a59efc55e19bf381f74cdf2ccdeeb2fcf73d3a49b8135049
+  metadata.gz: 28621b0b5ed052c16667b41c8398a0533d16b631cc31944a4d48f0f468da34a798e603b568435579afd1b07dae2a11ee9a7276d6b31f2a37baf9212aab5fa80a
+  data.tar.gz: 8740278135b296f312a8ef5189ab25d95c3e6428f27c286b7d10d5d359197e52280682c9adeed0305859663f6d90b4d5596465274d3eb54e5c0893326d305110

data/lib/no_fly_list/railties/tasks.rake

@@ -20,7 +20,7 @@ namespace :no_fly_list do
 
     classes.each do |klass|
       color = NoFlyList::TaskHelpers.adapter_color(klass)
-      type = klass.included_modules.include?(NoFlyList::ApplicationTag) ? 'Global' : 'Model-specific'
+      type = klass.included_modules.include?(NoFlyList::ApplicationTag) ? "Global" : "Model-specific"
 
       puts "#{color}#{klass.name}#{NoFlyList::TaskHelpers::COLORS[:reset]}"
       puts "  Type: #{type}"
@@ -42,8 +42,8 @@ namespace :no_fly_list do
       puts "  #{message}"
 
       [
-        ["#{klass.name}Tag", "Tags", :tag],
-        ["#{klass.name}::Tagging", "Taggings", :tagging]
+        [ "#{klass.name}Tag", "Tags", :tag ],
+        [ "#{klass.name}::Tagging", "Taggings", :tagging ]
       ].each do |class_name, type, column_type|
         if (check_class = NoFlyList::TaskHelpers.check_class(class_name))
           status, message = NoFlyList::TaskHelpers.check_table(check_class)
@@ -53,13 +53,13 @@ namespace :no_fly_list do
             puts "  #{NoFlyList::TaskHelpers.format_columns(check_class)}"
           end
         else
-          puts "  #{NoFlyList::TaskHelpers::colorize('✗', :red)} #{type} class not found: #{class_name}"
+          puts "  #{NoFlyList::TaskHelpers.colorize('✗', :red)} #{type} class not found: #{class_name}"
         end
       end
 
       klass._no_fly_list.tag_contexts.each do |context, config|
         puts "\n  Context: #{context}"
-        bullet = NoFlyList::TaskHelpers::colorize('•', :green)
+        bullet = NoFlyList::TaskHelpers.colorize("•", :green)
         puts "  #{bullet} Tag class: #{config[:tag_class_name]}"
         puts "  #{bullet} Tagging class: #{config[:tagging_class_name]}"
         puts "  #{bullet} Polymorphic: #{config[:polymorphic]}"

data/lib/no_fly_list/tagging_proxy.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 module NoFlyList
   class TaggingProxy
     include Enumerable
@@ -30,14 +28,15 @@ module NoFlyList
       @transformer = transformer.is_a?(String) ? transformer.constantize : transformer
       @restrict_to_existing = restrict_to_existing
       @limit = limit
-      @pending_changes = []
+      @pending_changes = nil # Use nil to indicate no changes yet
+      @clear_operation = false
     end
 
     # Determines if tags have changed from database state
     # @return [Boolean] True if pending changes differ from database
     # @api private
     def changed?
-      @pending_changes.present? && @pending_changes != current_list_from_database
+      @clear_operation || (!@pending_changes.nil? && @pending_changes != current_list_from_database)
     end
 
     def method_missing(method_name, *args)
@@ -73,7 +72,7 @@ module NoFlyList
 
     # @return [Boolean] true if the proxy is valid
     def save
-      return true unless @pending_changes.any?
+      return true unless changed?
       return false unless valid?
 
       # Prevent recursive validation
@@ -92,8 +91,9 @@ module NoFlyList
 
           # Update counter
           model.update_column("#{@context}_count", 0) if setup[:counter_cache]
+
           # Create new tags
-          @pending_changes.each do |tag_name|
+          pending_list.each do |tag_name|
             tag = find_or_create_tag(tag_name)
             next unless tag
 
@@ -112,7 +112,7 @@ module NoFlyList
           end
         end
         # Update counter to match the actual count
-        model.update_column("#{@context}_count", @pending_changes.size) if setup[:counter_cache]
+        model.update_column("#{@context}_count", pending_list.size) if setup[:counter_cache]
 
         refresh_from_database
         true
@@ -133,15 +133,22 @@ module NoFlyList
 
     # @return [Integer]
     def count
+      # Always return the database count for count operations
       @model.send(@context.to_s).count
     end
 
     # @return [Integer]
     def size
-      if @pending_changes.any?
+      # For size, return the database count if we've had a validation error
+      if !valid?
+        count
+        # Otherwise show pending changes
+      elsif @clear_operation
+        0
+      elsif !@pending_changes.nil?
         @pending_changes.size
       else
-        @model.send(@context.to_s).size
+        count
       end
     end
 
@@ -160,9 +167,44 @@ module NoFlyList
       @transformer_name ||= transformer.name
     end
 
+    # Returns tags that will be added (not in database but in pending changes)
+    # @return [Array<String>] Tags to be added
+    def additions
+      return [] if @clear_operation
+      return [] if @pending_changes.nil?
+
+      @pending_changes - current_list_from_database
+    end
+
+    # Returns tags that will be removed (in database but not in pending changes)
+    # @return [Array<String>] Tags to be removed
+    def removals
+      if @clear_operation
+        current_list_from_database
+      elsif @pending_changes.nil?
+        []
+      else
+        current_list_from_database - @pending_changes
+      end
+    end
+
     # @return [String]
     def inspect
-      "#<#{self.class.name} tags=#{current_list.inspect} transformer_with=#{transformer_name} >"
+      if @clear_operation
+        db_tags = current_list_from_database
+        "#<#{self.class.name} tags=[] changes=[CLEARING ALL (#{db_tags.size}): #{db_tags.inspect}] transformer_with=#{transformer_name}>"
+      elsif !@pending_changes.nil?
+        add_list = additions
+        remove_list = removals
+        changes = []
+        changes << "+#{add_list.inspect}" if add_list.any?
+        changes << "-#{remove_list.inspect}" if remove_list.any?
+        changes_str = changes.join(", ")
+
+        "#<#{self.class.name} tags=#{current_list.inspect} changes=[#{changes_str}] transformer_with=#{transformer_name}>"
+      else
+        "#<#{self.class.name} tags=#{current_list.inspect} transformer_with=#{transformer_name}>"
+      end
     end
 
     # Adds one or more tags to the current tag list
@@ -174,6 +216,7 @@ module NoFlyList
     def add(*tags)
       return self if limit_reached?
 
+      @clear_operation = false
       new_tags = if tags.size == 1 && tags.first.is_a?(String)
                    transformer.parse_tags(tags.first)
       else
@@ -181,8 +224,12 @@ module NoFlyList
       end
       return self if new_tags.empty?
 
-      @pending_changes = current_list + new_tags
+      # Initialize @pending_changes with database values if not yet initialized
+      @pending_changes = current_list_from_database if @pending_changes.nil?
+
+      @pending_changes = @pending_changes + new_tags
       @pending_changes.uniq!
+      mark_record_dirty
       self
     end
 
@@ -199,13 +246,20 @@ module NoFlyList
     # @return [TaggingProxy] Returns self for method chaining
     # @raise [ActiveRecord::RecordInvalid] If validation fails
     def remove(*tags)
-      old_list = current_list.dup
+      @clear_operation = false
+
+      # Initialize @pending_changes with database values if not yet initialized
+      @pending_changes = current_list_from_database if @pending_changes.nil?
+
+      old_list = @pending_changes.dup
+
       tags_to_remove = if tags.size == 1 && tags.first.is_a?(String)
                          transformer.parse_tags(tags.first)
       else
                          tags.flatten.map { |tag| tag.to_s.strip }
       end
-      @pending_changes = current_list - tags_to_remove
+
+      @pending_changes = @pending_changes - tags_to_remove
       mark_record_dirty if @pending_changes != old_list
       self
     end
@@ -220,9 +274,9 @@ module NoFlyList
     # @example Clear all tags
     #   tags.clear #=> []
     def clear
-      old_list = current_list.dup
+      @clear_operation = true
       @pending_changes = []
-      mark_record_dirty if @pending_changes != old_list
+      mark_record_dirty if current_list_from_database.any?
       model.write_attribute("#{@context}_count", 0) if setup[:counter_cache]
       self
     end
@@ -235,6 +289,7 @@ module NoFlyList
     def clear!
       @model.send(@context.to_s).destroy_all
       @pending_changes = []
+      @clear_operation = false
       @model.update_column("#{@context}_count", 0) if setup[:counter_cache]
       self
     end
@@ -276,7 +331,9 @@ module NoFlyList
     end
 
     def set_list(_context, value)
+      @clear_operation = false
       @pending_changes = transformer.parse_tags(value)
+      mark_record_dirty
       valid? # Just check validity without raising
       self
     end
@@ -286,24 +343,25 @@ module NoFlyList
     end
 
     def refresh_from_database
-      @pending_changes = []
+      @pending_changes = nil
+      @clear_operation = false
     end
 
     def validate_limit
       return unless @limit
-      return if @pending_changes.size <= @limit
+      return if pending_list.size <= @limit
 
-      errors.add(:base, "Cannot have more than #{@limit} tags (attempting to save #{@pending_changes.size})")
+      errors.add(:base, "Cannot have more than #{@limit} tags (attempting to save #{pending_list.size})")
     end
 
     def validate_existing_tags
       return unless @restrict_to_existing
-      return if @pending_changes.empty?
+      return if pending_list.empty?
 
       # Transform tags to lowercase for comparison
-      normalized_changes = @pending_changes.map(&:downcase)
+      normalized_changes = pending_list.map(&:downcase)
       existing_tags = @tag_model.where("LOWER(name) IN (?)", normalized_changes).pluck(:name)
-      missing_tags = @pending_changes - existing_tags
+      missing_tags = pending_list - existing_tags
 
       return unless missing_tags.any?
 
@@ -316,9 +374,9 @@ module NoFlyList
 
     def setup
       @setup ||= begin
-        context = @context.to_sym
-        @model.class._no_fly_list.tag_contexts[context]
-      end
+                   context = @context.to_sym
+                   @model.class._no_fly_list.tag_contexts[context]
+                 end
     end
 
     def find_or_create_tag(tag_name)
@@ -329,36 +387,24 @@ module NoFlyList
       end
     end
 
-    def save_changes
-      # Clear existing tags
-      model.send(context_taggings).delete_all
-
-      # Create new tags
-      @pending_changes.each do |tag_name|
-        tag = find_or_create_tag(tag_name)
-        next unless tag
-
-        attributes = {
-          tag: tag,
-          context: @context.to_s.singularize
-        }
-
-        # Add polymorphic attributes for polymorphic tags
-        if setup[:polymorphic]
-          attributes[:taggable_type] = model.class.name
-          attributes[:taggable_id] = model.id
-        end
-
-        # Use create! to ensure we catch any errors
-        model.send(context_taggings).create!(attributes)
+    # Helper method to get the list of tags that should be saved
+    def pending_list
+      if @clear_operation
+        []
+      elsif !@pending_changes.nil?
+        @pending_changes
+      else
+        current_list_from_database
       end
-
-      refresh_from_database
-      true
     end
 
     def current_list
-      if @pending_changes.any?
+      # If validation failed, always return what's in the database
+      if errors.any?
+        current_list_from_database
+      elsif @clear_operation
+        []
+      elsif !@pending_changes.nil?
         @pending_changes
       else
         current_list_from_database

data/lib/no_fly_list/task_helpers.rb

@@ -13,8 +13,8 @@ module NoFlyList
     }.freeze
 
     REQUIRED_COLUMNS = {
-      tag: ['name'],
-      tagging: ['tag_id', 'taggable_id', 'context']
+      tag: [ "name" ],
+      tagging: %w[tag_id taggable_id context]
     }.freeze
 
     def self.adapter_color(klass)
@@ -28,9 +28,9 @@ module NoFlyList
 
     def self.check_table(klass)
       klass.table_exists?
-      [true, "#{colorize('✓', :green)} Table exists: #{klass.table_name}"]
+      [ true, "#{colorize('✓', :green)} Table exists: #{klass.table_name}" ]
     rescue StandardError => e
-      [false, "#{colorize('✗', :red)} Error: #{e.message}"]
+      [ false, "#{colorize('✗', :red)} Error: #{e.message}" ]
     end
 
     def self.verify_columns(klass, type)
@@ -61,16 +61,15 @@ module NoFlyList
     def self.find_taggable_classes
       Rails.application.eager_load!
       ActiveRecord::Base.descendants.select do |klass|
-        klass.included_modules.any? { |mod| mod.in?([NoFlyList::TaggableRecord]) }
+        klass.included_modules.any? { |mod| mod.in?([ NoFlyList::TaggableRecord ]) }
       end
     end
 
     def self.find_tag_classes
       Rails.application.eager_load!
       ActiveRecord::Base.descendants.select do |klass|
-        klass.included_modules.any? { |mod| mod.in?([NoFlyList::ApplicationTag, NoFlyList::TagRecord]) }
+        klass.included_modules.any? { |mod| mod.in?([ NoFlyList::ApplicationTag, NoFlyList::TagRecord ]) }
       end
     end
   end
 end
-

data/lib/no_fly_list/version.rb

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

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: no_fly_list
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Abdelkader Boudih
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-20 00:00:00.000000000 Z
+date: 2025-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -67,7 +66,6 @@ licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -82,8 +80,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 3.6.2
 specification_version: 4
 summary: Tagging system for ActiveRecord models
 test_files: []