no_fly_list 0.7.2 → 0.7.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9fb7865780acd9586b0ff700686a1546a4c294f9fc3d7a71fe47f2fb7a6def0c
-  data.tar.gz: be5186e8116c5db42c4aa4e3536ab4343b95967f5cd05dcfb64c8f99ebb85270
+  metadata.gz: 5faaa3eaa7a8bc8f8186f8ee0dd10971f0c187963a4b9ef93cf2ef00963bedc0
+  data.tar.gz: 575274c475a0a13b4b94194f904aeec4e4d6d7795cd36db6b12ad6eff28a64f3
 SHA512:
-  metadata.gz: 630d2aa00cc4b2465321d1040697cff6c54124247275a0b75677d70653fe347ec1c0ad7e9c573dea060de07be1de02977f1ff6f467bfe1465299a1f764e35325
-  data.tar.gz: 1201844673a3893a3c760ab1fc88ed6826839d799bca6a93b17c2c3e802fa4e9df4d485bdc5511cec4c7cbf2bdcc3cba9cfd1280970d58b5eaa93492f7ece67a
+  metadata.gz: a98397336ebfc91db71338a108df13b48af0fdc1287c2c3e9ce98569394fc05cfe8b4450d1ef2272c15b7a077966b21671c6817c4aa2ae55972f4683545f5032
+  data.tar.gz: 1cb1ec3ffcfb380fd616dabf5e5551e5ec5c5f52f24ab966aa68542c6f60f4770b293bdba8ee5841583e93b7a9d8a20a7ef25a02627473382ad2c84baaa9f88c

data/lib/no_fly_list/taggable_record.rb

@@ -59,7 +59,12 @@ module NoFlyList
 
           proxy = instance_variable_get(var)
           next if proxy.nil?
-          return false unless proxy.save
+          unless proxy.save
+            proxy.errors.each do |error|
+              errors.add(:base, error.message)
+            end
+            throw :abort
+          end
         end
         true
       ensure

data/lib/no_fly_list/tagging_proxy.rb

@@ -332,11 +332,12 @@ module NoFlyList
 
     def current_list_from_database
       if setup[:polymorphic]
-        tagging_table = setup[:tagging_class_name].tableize
+        tagging_klass = setup[:tagging_class_name].constantize
+        tagging_table = tagging_klass.arel_table
+
         @model.send(@context.to_s)
-              .joins("INNER JOIN #{tagging_table} ON #{tagging_table}.tag_id = tags.id")
-              .where("#{tagging_table}.taggable_type = ? AND #{tagging_table}.taggable_id = ?",
-                     @model.class.name, @model.id)
+              .where(tagging_table[:taggable_type].eq(@model.class.name))
+              .where(tagging_table[:taggable_id].eq(@model.id))
               .pluck(:name)
       else
         @model.send(@context.to_s).pluck(:name)
@@ -345,7 +346,7 @@ module NoFlyList
 
     def set_list(_context, value)
       @clear_operation = false
-      @pending_changes = transformer.parse_tags(value)
+      @pending_changes = transformer.parse_tags(value).uniq.reject(&:blank?)
       mark_record_dirty
       valid? # Just check validity without raising
       self

data/lib/no_fly_list/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: no_fly_list
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
 platform: ruby
 authors:
 - Abdelkader Boudih
@@ -23,7 +23,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '7.2'
-description: Tagging system for ActiveRecord models inspired by the TSA
+description: A flexible, high-performance tagging system for Rails applications with
+  support for polymorphic tags, custom transformers, and database-specific optimizations
 email:
 - terminale@gmail.com
 executables: []
@@ -66,7 +67,12 @@ homepage: https://github.com/contriboss/no_fly_list
 licenses:
 - MIT
 metadata:
+  homepage_uri: https://github.com/contriboss/no_fly_list
+  source_code_uri: https://github.com/contriboss/no_fly_list
+  changelog_uri: https://github.com/contriboss/no_fly_list/blob/master/CHANGELOG.md
+  bug_tracker_uri: https://github.com/contriboss/no_fly_list/issues
   rubygems_mfa_required: 'true'
+  github_repo: ssh://github.com/contriboss/no_fly_list
 rdoc_options: []
 require_paths:
 - lib
@@ -83,5 +89,5 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 rubygems_version: 3.6.9
 specification_version: 4
-summary: Tagging system for ActiveRecord models
+summary: Modern tagging system for Rails 7.2+ applications
 test_files: []