attr_json 2.3.0 → 2.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eb4f8078039ec81cc25e7a95318ff5e42cb5bf85a955b40326f60f9022af16a2
-  data.tar.gz: 635f08ce5fa34ce45dfdd5256ae0e81fdcc2d455f4525bad5f71f5b0c4391c6b
+  metadata.gz: 5a2727136aa434d698d6c54612f529c3765c2603012725161850be4cfd7f317f
+  data.tar.gz: 0da797831a4a0a7c904c15b31bdb37dff2afaec49ae35caec993f85cfd23a31e
 SHA512:
-  metadata.gz: 22c35f674c7b5f3a4e91f348d20439df3e8fe175ca5d3c8938720a5e71739df2474cdddd5fdab5f0527f8751ed0fb6dee11ba7f75889c143daca92bd84afc86b
-  data.tar.gz: 636628f0083dd61c869de5b9eb65a0c11988560e26d5dcd8e822759788bdac4bdb3c6f0359e2257867ac91a61721fda32df54a36f51a001450ef883289be433a
+  metadata.gz: a7e88ae65b9a417c00cee127fa8a90c46566698bd3835a74d5de941bbc9a64e0fdce9af72c1dd6adbb3fda29e02bc0817fccfc6b419b75088f1666e8a7c8ddc8
+  data.tar.gz: 9bad39be885c3e8ce0065259713c17aebb96cb498098d25540ef3cc8239b56fc41a616f34353d4d21f1eea103d6ff87a084a36ce8812769389daf4598acc0c1a

data/.github/workflows/ci.yml

@@ -62,6 +62,9 @@ jobs:
           - gemfile: rails_7_1
             ruby: 3.2
 
+          - gemfile: rails_7_1
+            ruby: 3.3
+
     name: ${{ matrix.gemfile }}, ruby ${{ matrix.ruby }}
 
     steps:

data/.github/workflows/future_rails_ci.yml

@@ -45,7 +45,7 @@ jobs:
         include:
 
           - gemfile: rails_edge
-            ruby: '3.1'
+            ruby: '3.3'
 
     name: ${{ matrix.gemfile }}, ruby ${{ matrix.ruby }}
 

data/CHANGELOG.md

@@ -4,7 +4,8 @@ Notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/jrochkind/attr_json/compare/v2.3.0...HEAD)
+
+## [Unreleased](https://github.com/jrochkind/attr_json/compare/v2.3.1...HEAD)
 
 ### Fixed
 
@@ -30,6 +31,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 *
 
+## [2.3.1](https://github.com/jrochkind/attr_json/compare/v2.3.0...2.3.1)
+
+### Fixed
+
+* Keep nested model validation working in Rails post-7.1, by no longer trying to re-use ActiveRecord::Validations::AssociatedValidator, instead supplying our own custom code. https://github.com/jrochkind/attr_json/pull/220
+
+* Validate PolymorphicModel instances. https://github.com/jrochkind/attr_json/pull/225 Thanks @LyricL-Gitster
+
+
 ## [2.3.0]((https://github.com/jrochkind/attr_json/compare/v2.2.0...v2.3.0))
 
 

data/lib/attr_json/model/nested_model_validator.rb

@@ -0,0 +1,27 @@
+module AttrJson
+  module Model
+    # Used to validate an attribute in an AttrJson::Model whose values are other models, when
+    # you want validation errors on the nested models to post up.
+    #
+    # This is based on ActiveRecord's own ActiveRecord::Validations::AssociatedValidator, and actually forked
+    # from it at https://github.com/rails/rails/blob/e37adfed4eff3b43350ec87222a922e9c72d9c1b/activerecord/lib/active_record/validations/associated.rb
+    #
+    # We used to simply use an ActiveRecord::Validations::AssociatedValidator, but as of https://github.com/jrochkind/attr_json/pull/220 (e1e798142d)
+    # it got ActiveRecord-specific functionality that no longer worked with our use case.
+    #
+    # No problem, the implementation is simple, we can provide it here, based on the last version that did work.
+    class NestedModelValidator < ActiveModel::EachValidator
+      def validate_each(record, attribute, value)
+        if Array(value).reject { |r| valid_object?(r) }.any?
+          record.errors.add(attribute, :invalid, **options.merge(value: value))
+        end
+      end
+
+      private
+        def valid_object?(record)
+          #(record.respond_to?(:marked_for_destruction?) && record.marked_for_destruction?) || record.valid?
+          record.valid?
+        end
+    end
+  end
+end

data/lib/attr_json/model.rb

@@ -8,6 +8,7 @@ require 'attr_json/attribute_definition/registry'
 
 require 'attr_json/type/model'
 require 'attr_json/model/cocoon_compat'
+require 'attr_json/model/nested_model_validator'
 
 require 'attr_json/serialization_coder_from_type'
 
@@ -207,8 +208,8 @@ module AttrJson
       # @option options [String,Symbol] :store_key (nil) Serialize to JSON using
       #   given store_key, rather than name as would be usual.
       #
-      # @option options [Boolean] :validate (true) Create an ActiveRecord::Validations::AssociatedValidator so
-      #   validation errors on the attributes post up to self.
+      # @option options [Boolean] :validate (true) Mak validation errors on the attributes post up
+      #   to self, using something similar to an ActiveRecord::Validations::AssociatedValidator
       def attr_json(name, type, **options)
         options.assert_valid_keys(*(AttributeDefinition::VALID_OPTIONS - [:container_attribute] + [:validate]))
 
@@ -219,10 +220,9 @@ module AttrJson
         )
 
         # By default, automatically validate nested models
-        if type.kind_of?(AttrJson::Type::Model) && options[:validate] != false
-          # Yes. we're passing an ActiveRecord::Validations validator, but
-          # it works fine for ActiveModel. If this changes in the future, tests will catch.
-          self.validates_with ActiveRecord::Validations::AssociatedValidator, attributes: [name.to_sym]
+        if (type.kind_of?(AttrJson::Type::Model) || type.kind_of?(AttrJson::Type::PolymorphicModel)) && options[:validate] != false
+          # Post validations up with something based on ActiveRecord::Validations::AssociatedValidator
+          self.validates_with ::AttrJson::Model::NestedModelValidator, attributes: [name.to_sym]
         end
 
         _attr_jsons_module.module_eval do

data/lib/attr_json/version.rb

@@ -1,3 +1,3 @@
 module AttrJson
-  VERSION = "2.3.0"
+  VERSION = "2.3.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: attr_json
 version: !ruby/object:Gem::Version
-  version: 2.3.0
+  version: 2.3.1
 platform: ruby
 authors:
 - Jonathan Rochkind
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-12-04 00:00:00.000000000 Z
+date: 2024-05-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -172,6 +172,7 @@ files:
 - lib/attr_json/config.rb
 - lib/attr_json/model.rb
 - lib/attr_json/model/cocoon_compat.rb
+- lib/attr_json/model/nested_model_validator.rb
 - lib/attr_json/nested_attributes.rb
 - lib/attr_json/nested_attributes/builder.rb
 - lib/attr_json/nested_attributes/multiparameter_attribute_writer.rb
@@ -207,7 +208,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.21
+rubygems_version: 3.4.19
 signing_key:
 specification_version: 4
 summary: ActiveRecord attributes stored serialized in a json column, super smooth.