attr_json 1.3.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 026a86cb550ed91faef1a0aa1eca9dd0fa1985d1d3ac3221da2a165d19145ef9
-  data.tar.gz: be400202cc1016c053ffcdced03d17706601141e402d8998da7c0a3d3cea4815
+  metadata.gz: 3f2658693e8b0c4d99e412e33c8d2ef67a70bb71302f0d3d079326bef2380c44
+  data.tar.gz: 6acbf7ca9e743a2a22550731baf495306bc087fd3787c02ea19435b94f96a85d
 SHA512:
-  metadata.gz: 98e164f55a903e7babe2a94cfb794d448391bfc75f080d9309f766764e15762f47f524aff7faa5933957302ad36af6f3b0e2ee0e323e0f32f20abbd8eb6e1e2f
-  data.tar.gz: 17e789b630a3a131a57cb8f5805a53641c9802f5e711cad38af73440aa37beb33f1f9a636479fcc63a5b892251c09408a7478746ceeb2df960d207945bea2668
+  metadata.gz: 9a3cf5a739623ba998162b0072b351a2ae07316691dc013c2f88820dc36d5119833d64bc89335379dbd636f65fcf4253f818649f57538c753b0a5d98fc7c5b6c
+  data.tar.gz: 79a977eb57509ad7035bb5d20e6212568d51b8062de11d536bbeae58acab8bc8db48eba181adeecff97286fa59042b6eb143359b41a7c65d73955fb18c2edc75

data/.github/workflows/ci.yml

@@ -53,10 +53,19 @@ jobs:
           - gemfile: rails_6_1
             ruby: 2.7
 
+          - gemfile: rails_6_1
+            ruby: '3.0'
+
+          - gemfile: rails_7_0
+            ruby: '3.0'
+
+          - gemfile: rails_7_0
+            ruby: 3.1
+
     name: ${{ matrix.gemfile }}, ruby ${{ matrix.ruby }}
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
 
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1

data/.github/workflows/future_rails_ci.yml

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

data/Appraisals

@@ -35,20 +35,28 @@ end
 appraise "rails-6-1" do
   gem 'combustion', "~> 1.0"
 
-  gem "rails", ">= 6.0.0.rc1", "< 6.2.0.a"
+  gem "rails", "~> 6.1.0"
   gem "pg", "~> 1.0"
+
+  # sprockets-rails is already a rails 6.1 dependency, but combustion is failing
+  # to require it, this is one way to get it required.
+  # https://github.com/pat/combustion/issues/128
+  gem "sprockets-rails"
 end
 
-appraise "rails-edge-6" do
-  # Edge rails needs unreleased combustion
-  # https://github.com/pat/combustion/issues/92
+appraise "rails-7-0" do
   gem 'combustion', "~> 1.0"
 
-  gem "rails", git: "https://github.com/rails/rails.git", branch: "master"
+  gem "rails", "~> 7.0.0"
   gem "pg", "~> 1.0"
+end
 
-  # We don't actually use coffeescript at all, we need coffee-rails as an explicit
-  # dependency just for transitory edge weirdness using current sprockets release
-  # with rails 6 edge.
-  gem 'coffee-rails'
+appraise "rails-edge" do
+  # need combustion edge to work with rails edge, will no longer
+  # be true on next combustion release, probably no later than Rails 7.1
+  # https://github.com/pat/combustion/pull/126
+  gem 'combustion', "~> 1.0", github: "pat/combustion"
+
+  gem "rails", git: "https://github.com/rails/rails.git", branch: "main"
+  gem "pg", "~> 1.0"
 end

data/CHANGELOG.md

@@ -4,7 +4,28 @@ 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/v1.3.0...HEAD)
+## [Unreleased](https://github.com/jrochkind/attr_json/compare/v1.4.0...HEAD)
+
+
+## [1.4.1](https://github.com/jrochkind/attr_json/compare/v1.4.0...v1.4.1)
+
+### Fixed
+
+* Fixed an obscure bug involving a conflict between attribute defaults and accepts_nested_attributes, in which defaults could overwrite assigned attributes. The `.fill_in_defaults` class method, which was never intended as public API and was commented accordingly, is gone. https://github.com/jrochkind/attr_json/pull/160
+
+## [1.4.0](https://github.com/jrochkind/attr_json/compare/v1.3.0...v1.4.0)
+
+### Changed
+
+* When using store_key feature on an AttrJson::Model, you should not be able to pass in the store_key as a key in initializer or assign_attributes. It was really a bug that this ended up mapped to attribute this way, which could cause a problem in some cases; but calling it out in Changed section because if you were accidentally relying on it, it may appear as a backwards incompat to you. https://github.com/jrochkind/attr_json/pull/125
+
+* Rails 7.0.0 allowed by gemspec and tested in CI
+
+### Fixed
+
+* polymorphic single type can be set to nil https://github.com/jrochkind/attr_json/pull/115
+* polymorphic models can be serialized from hash in container attribute. Thanks @machty. https://github.com/jrochkind/attr_json/pull/123
+* fix bug with deserialization of nested attributes that have types that apply different serialization vs cast logic. Thanks @bradleesand. https://github.com/jrochkind/attr_json/pull/125
 
 ## [1.3.0](https://github.com/jrochkind/attr_json/compare/v1.2.0...v1.3.0)
 

data/Gemfile

@@ -18,17 +18,24 @@ gem 'combustion', '~> 1.1'
 # be the way to do it.
 gem 'rails'
 
-# We should not really need to mention railties, it's already a dependency of
-# rails, but seems to be necessary to get around some mystery bug in bundler
-# dependency resolution.
-gem 'railties'
-
 gem "pg"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem 'cocoon', ">= 1.2"
 gem 'jquery-rails'
 
+# Even though we don't use coffee-script, when running specs, some part of rails
+# or other part of our stack is still insisting on requiring it, for reasons we
+# don't understand, so we need to depend on it.
+gem "coffee-rails"
+
+# We do some tests using cocoon via sprockets, which needs sprockets-rails,
+# which is not automatically available in Rails 7. We add it explicitly,
+# which will duplciate dependences in rails pre-7, but add for rails 7. Not sure
+# the future of cocoon in general. https://github.com/nathanvda/cocoon/issues/555
+gem "sprockets-rails"
+
+
 gem 'capybara', "~> 3.0"
 gem 'webdrivers', '~> 4.0'
 gem "selenium-webdriver"

data/README.md

@@ -398,27 +398,29 @@ Rails 5.1+ on `attr_json`s on your ActiveRecord classes that include
 `AttrJson::Record`, by including `AttrJson::Record::Dirty`.
 Change-tracking methods are available off the `attr_json_changes` method.
 
-    class MyModel < ActiveRecord::Base
-       include AttrJson::Record
-       include AttrJson::Record::Dirty
-
-       attr_json :str, :string
-    end
-
-    model = MyModel.new
-    model.str = "old"
-    model.save
-    model.str = "new"
+```ruby
+class MyModel < ActiveRecord::Base
+   include AttrJson::Record
+   include AttrJson::Record::Dirty
 
-    # All and only "new" style dirty tracking methods (Raisl 5.1+)
-    # are available:
+   attr_json :str, :string
+end
 
-    model.attr_json_changes.saved_changes
-    model.attr_json_changes.changes_to_save
-    model.attr_json_changes.saved_change_to_str?
-    model.attr_json_changes.saved_change_to_str
-    model.attr_json_changes.will_save_change_to_str?
-    # etc
+model = MyModel.new
+model.str = "old"
+model.save
+model.str = "new"
+
+# All and only "new" style dirty tracking methods (Raisl 5.1+)
+# are available:
+
+model.attr_json_changes.saved_changes
+model.attr_json_changes.changes_to_save
+model.attr_json_changes.saved_change_to_str?
+model.attr_json_changes.saved_change_to_str
+model.attr_json_changes.will_save_change_to_str?
+# etc
+```
 
 More options are available, including merging changes from 'ordinary'
 ActiveRecord attributes in. See docs on [Dirty Tracking](./doc_src/dirty_tracking.md)
@@ -476,11 +478,15 @@ to prevent overwriting other updates from processes.
 
 ## State of Code, and To Be Done
 
-The functionality that is here seems pretty solid, and is being used by jrochkind in a production app.
+This code is solid and stable and is being used in production by at least a handful of people, including the primary maintainer, jrochkind. 
+
+The project is currently getting very little maintainance -- and is still working reliably through Rails releases. It is tested on edge rails and ruby (and has needed very few if any changes with subsequent releases), and I endeavor to keep it working as Rails keeps releasing. 
+
+In order to keep the low-maintenace scenario sustainable, I am *very* cautious accepting new features, especially if they increase code complexity at all. Even if you have a working PR, I may be reluctant to accept it. I'm prioritizing sustainability and stability over new features, and so far this is working out well. However, discussion is always welcome! Especially when paired with code (failing tests for the bugfix or feature you want are super helpful on their own!). 
 
 We are committed to [semantic versioning](https://semver.org/) and will endeavor to release no backwards breaking changes without a major version. We are also serious about minimizing backwards incompat releases altogether (ie minimiing major version releases).
 
-Feedback of any kind of _very welcome_, please feel free to use the issue tracker.
+Feedback of any kind of _very welcome_, please feel free to use the issue tracker. It is hard to get a sense of how many people are actually using this, which is helpful both for my own sense of reward and for anyone to get a sense of the size of the userbase -- feel free to say hi and let us know how you are using it! 
 
 Except for the jsonb_contains stuff using postgres jsonb contains operator, I don't believe any postgres-specific features are used. It ought to work with MySQL, testing and feedback welcome. (Or a PR to test on MySQL?).  My own interest is postgres.
 
@@ -488,26 +494,30 @@ This is still mostly a single-maintainer operation, so has all the sustainabilit
 
 ### Possible future features:
 
-* partial updates for json hashes would be really nice: Using postgres jsonb merge operators to only overwrite what changed. In my initial attempts, AR doesn't make it easy to customize this.
+* Make AttrJson::Model lean more heavily on ActiveModel::Attributes API that did not fully exist in first version of attr_json [hope for a future attr_json 2.0 release]
+
+* Make AttrJson::Record insist on creating rails cover attributes (no longer optional)  and integrating more fully into rails, including rails dirty tricking, eliminating need for custom dirty tracking implementation. Overall decrease in lines of code. Can use jsonb_accessor as guide for some aspects. [hope for inclusion in future attr_json 2.0 release]
+
+* partial updates for json hashes would be really nice: Using postgres jsonb merge operators to only overwrite what changed. In my initial attempts, AR doesn't make it easy to customize this. [update: this is hard, probably not coming soon]
 
-* seamless compatibility with ransack
+* seamless compatibility with ransack [update: not necessarily prioritized]
 
 * Should we give AttrJson::Model a before_serialize hook that you might
   want to use similar to AR before_save?  Should AttrJson::Models
-  raise on trying to serialize an invalid model?
+  raise on trying to serialize an invalid model? [update: eh, hasn't really come up]
 
 * There are limits to what you can do with just jsonb_contains
   queries. We could support operations like `>`, `<`, `<>`
   as [jsonb_accessor](https://github.com/devmynd/jsonb_accessor),
   even accross keypaths. (At present, you could use a
   before_savee to denormalize/renormalize copy your data into
-  ordinary AR columns/associations for searching. Or perhaps a postgres ts_vector for text searching. Needs to be worked out.)
+  ordinary AR columns/associations for searching. Or perhaps a postgres ts_vector for text searching. Needs to be worked out.) [update: interested, but not necessarily prioritized. This one would be interesting for a third-party PR draft!]
 
 * We could/should probably support `jsonb_order` clauses, even
-  accross key paths, like jsonb_accessor.
+  accross key paths, like jsonb_accessor. [update: interested but not necessarily prioritized]
 
 * Could we make these attributes work in ordinary AR where, same
-  as they do in jsonb_contains? Maybe.
+  as they do in jsonb_contains? Maybe. [update: probably not]
 
 ## Development
 

data/attr_json.gemspec

@@ -44,14 +44,30 @@ attributes use as much of the existing ActiveRecord architecture as we can.}
 
   # Only to get CI to work on versions of Rails other than we release with,
   # should never release a gem with RAILS_GEM set!
-  unless ENV['APPRAISAL_INITIALIZED'] || ENV["TRAVIS"]
-    spec.add_runtime_dependency "activerecord", ">= 5.0.0", "< 6.2"
+  unless ENV['APPRAISAL_INITIALIZED'] || ENV["TRAVIS"] || ENV['CI']
+    spec.add_runtime_dependency "activerecord", ">= 5.0.0", "< 7.1"
   end
 
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake", ">= 10.0"
   spec.add_development_dependency "rspec", "~> 3.7"
-  spec.add_development_dependency "database_cleaner", "~> 1.5"
   spec.add_development_dependency "yard-activesupport-concern"
   spec.add_development_dependency "appraisal", "~> 2.2"
+
+  # Working around annoying issue in selenium 3.x with ruby 3.0.
+  # we don't actually use rexml ourselves. selenium 3 is a dependency
+  # of webdrivers, and tries to use rexml without depending on it
+  # as is needed in ruby 3.
+  #
+  # https://github.com/SeleniumHQ/selenium/issues/9001
+  #
+  # if in the future you can remove this dependecy and still have tests pass
+  # under ruby 3.x, you're good.
+  spec.add_development_dependency "rexml"
+
+  # Used only for Capybara.server in our spec_helper.rb.
+  # webrick is no longer included in ruby 3.0, so has to
+  # be expressed as a dependecy, unless we switch
+  # capybara to use alternate webserver.
+  spec.add_development_dependency "webrick", "~> 1.0"
 end

data/gemfiles/rails_5_0.gemfile

@@ -4,12 +4,13 @@ source "https://rubygems.org"
 
 gem "combustion", "~> 0.9.0"
 gem "rails", "~> 5.0.0"
-gem "railties"
 gem "pg", "~> 0.18"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"

data/gemfiles/rails_5_1.gemfile

@@ -4,12 +4,13 @@ source "https://rubygems.org"
 
 gem "combustion", "~> 0.9.0"
 gem "rails", "~> 5.1.0"
-gem "railties"
 gem "pg", "~> 1.0"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"

data/gemfiles/rails_5_2.gemfile

@@ -4,12 +4,13 @@ source "https://rubygems.org"
 
 gem "combustion", "~> 0.9.0"
 gem "rails", "~> 5.2.0"
-gem "railties"
 gem "pg", "~> 1.0"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"

data/gemfiles/rails_6_0.gemfile

@@ -4,12 +4,13 @@ source "https://rubygems.org"
 
 gem "combustion", "~> 1.0"
 gem "rails", ">= 6.0.0", "< 6.1"
-gem "railties"
 gem "pg", "~> 1.0"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"

data/gemfiles/rails_6_1.gemfile

@@ -3,13 +3,14 @@
 source "https://rubygems.org"
 
 gem "combustion", "~> 1.0"
-gem "rails", "~> 6.1"
-gem "railties"
+gem "rails", "~> 6.1.0"
 gem "pg", "~> 1.0"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"

data/gemfiles/{rails_edge_6.gemfile → rails_7_0.gemfile}

@@ -3,17 +3,17 @@
 source "https://rubygems.org"
 
 gem "combustion", "~> 1.0"
-gem "rails", git: "https://github.com/rails/rails.git", branch: "master"
-gem "railties"
+gem "rails", "~> 7.0.0"
 gem "pg", "~> 1.0"
 gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"
 gem "byebug"
-gem "coffee-rails"
 
 gemspec path: "../"

data/gemfiles/rails_edge.gemfile

@@ -2,18 +2,18 @@
 
 source "https://rubygems.org"
 
-gem "combustion", "~> 1.0"
-gem "rails", git: "https://github.com/rails/rails.git", branch: "master"
-gem "railties"
+gem "combustion", "~> 1.0", github: "pat/combustion"
+gem "rails", git: "https://github.com/rails/rails.git", branch: "main"
 gem "pg", "~> 1.0"
-gem "rspec-rails", "~> 3.7"
+gem "rspec-rails", "~> 4.0"
 gem "simple_form", ">= 4.0"
 gem "cocoon", ">= 1.2"
 gem "jquery-rails"
+gem "coffee-rails"
+gem "sprockets-rails"
 gem "capybara", "~> 3.0"
 gem "webdrivers", "~> 4.0"
 gem "selenium-webdriver"
 gem "byebug"
-gem "coffee-rails"
 
 gemspec path: "../"

data/lib/attr_json/model.rb

@@ -103,16 +103,27 @@ module AttrJson
       end
 
 
-      # Like `.new`, but translate store keys in hash
+      # The inverse of model#serializable_hash -- re-hydrates a serialized hash to a model.
+      #
+      # Similar to `.new`, but translates things that need to be translated in deserialization,
+      # like store_keys, and properly calling deserialize on the underlying types.
+      #
+      # @example Model.new_from_serializable(hash)
       def new_from_serializable(attributes = {})
-        attributes = attributes.transform_keys do |key|
+        attributes = attributes.collect do |key, value|
           # store keys in arguments get translated to attribute names on initialize.
           if attribute_def = self.attr_json_registry.store_key_lookup("", key.to_s)
-            attribute_def.name.to_s
-          else
-            key
+            key = attribute_def.name.to_s
           end
-        end
+
+          attr_type = self.attr_json_registry.has_attribute?(key) && self.attr_json_registry.type_for_attribute(key)
+          if attr_type
+            value = attr_type.deserialize(value)
+          end
+
+          [key, value]
+        end.to_h
+
         self.new(attributes)
       end
 
@@ -166,27 +177,6 @@ module AttrJson
         end
       end
 
-      # This should kind of be considered 'protected', but the semantics
-      # of how we want to call it don't give us a visibility modifier that works.
-      # Prob means refactoring called for. TODO?
-      def fill_in_defaults(hash)
-        # Only if we need to mutate it to add defaults, we'll dup it first. deep_dup not neccesary
-        # since we're only modifying top-level here.
-        duped = false
-        attr_json_registry.definitions.each do |definition|
-          if definition.has_default? && ! (hash.has_key?(definition.store_key.to_s) || hash.has_key?(definition.store_key.to_sym))
-            unless duped
-              hash = hash.dup
-              duped = true
-            end
-
-            hash[definition.store_key] = definition.provide_default!
-          end
-        end
-
-        hash
-      end
-
       private
 
       # Define an anonymous module and include it, so can still be easily
@@ -202,11 +192,9 @@ module AttrJson
     end
 
     def initialize(attributes = {})
-      if !attributes.respond_to?(:transform_keys)
-        raise ArgumentError, "When assigning attributes, you must pass a hash as an argument."
-      end
+      super
 
-      super(self.class.fill_in_defaults(attributes))
+      fill_in_defaults!
     end
 
     def attributes
@@ -292,6 +280,14 @@ module AttrJson
 
     private
 
+    def fill_in_defaults!
+      self.class.attr_json_registry.definitions.each do |definition|
+        if definition.has_default? && !attributes.has_key?(definition.name.to_s)
+          self.send("#{definition.name.to_s}=", definition.provide_default!)
+        end
+      end
+    end
+
     def _attr_json_write(key, value)
       if attribute_def = self.class.attr_json_registry[key.to_sym]
         attributes[key.to_s] = attribute_def.cast(value)

data/lib/attr_json/record/dirty.rb

@@ -270,7 +270,13 @@ module AttrJson
         # find it from currently declared attributes.
         # https://github.com/rails/rails/blob/6aa5cf03ea8232180ffbbae4c130b051f813c670/activemodel/lib/active_model/attribute_methods.rb#L463-L468
         def matched_attribute_method(method_name)
-          matches = self.class.send(:attribute_method_matchers_matching, method_name)
+          if self.class.respond_to?(:attribute_method_patterns_matching, true)
+            # Rails 7.1+
+            matches = self.class.send(:attribute_method_patterns_matching, method_name)
+          else
+            matches = self.class.send(:attribute_method_matchers_matching, method_name)
+          end
+
           matches.detect do |match|
             registry.has_attribute?(match.attr_name)
           end

data/lib/attr_json/type/model.rb

@@ -32,10 +32,10 @@ module AttrJson
           # to_hash is actually the 'implicit' conversion, it really is a hash
           # even though it isn't is_a?(Hash), try to_hash first before to_h,
           # the explicit conversion.
-          model.new_from_serializable(v.to_hash)
+          model.new(v.to_hash)
         elsif v.respond_to?(:to_h)
           # TODO Maybe we ought not to do this on #to_h?
-          model.new_from_serializable(v.to_h)
+          model.new(v.to_h)
         elsif model.attr_json_config.bad_cast == :as_nil
           # This was originally default behavior, to be like existing ActiveRecord
           # which kind of silently does this for non-castable basic values. That
@@ -58,7 +58,31 @@ module AttrJson
       end
 
       def deserialize(v)
-        cast(v)
+        if v.nil?
+          # important to stay nil instead of empty object, because they
+          # are different things.
+          v
+        elsif v.kind_of? model
+          v
+        elsif v.respond_to?(:to_hash)
+          # to_hash is actually the 'implicit' conversion, it really is a hash
+          # even though it isn't is_a?(Hash), try to_hash first before to_h,
+          # the explicit conversion.
+          model.new_from_serializable(v.to_hash)
+        elsif v.respond_to?(:to_h)
+          # TODO Maybe we ought not to do this on #to_h? especially here in deserialize?
+          model.new_from_serializable(v.to_h)
+        elsif model.attr_json_config.bad_cast == :as_nil
+          # TODO should we have different config value for bad_deserialize vs bad_cast?
+
+          # This was originally default behavior, to be like existing ActiveRecord
+          # which kind of silently does this for non-castable basic values. That
+          # ended up being confusing in the basic case, so now we raise by default,
+          # but this is still configurable.
+          nil
+        else
+          raise BadCast.new("Can not cast from #{v.inspect} to #{self.type}")
+        end
       end
 
       # these guys are definitely mutable, so we need this.

data/lib/attr_json/type/polymorphic_model.rb

@@ -114,6 +114,11 @@ module AttrJson
       end
 
       def serialize(v)
+        return nil if v.nil?
+
+        # if it's not already a model cast it to a model if possible (eg it's a hash)
+        v = cast(v)
+
         model_name = v.class.name
         type = type_for_model_name(model_name)
 

data/lib/attr_json/version.rb

@@ -1,3 +1,3 @@
 module AttrJson
-  VERSION = "1.3.0"
+  VERSION = "1.4.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: attr_json
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.1
 platform: ruby
 authors:
 - Jonathan Rochkind
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-12-13 00:00:00.000000000 Z
+date: 2022-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +19,7 @@ dependencies:
         version: 5.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.2'
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +29,7 @@ dependencies:
         version: 5.0.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6.2'
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -73,21 +73,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '3.7'
 - !ruby/object:Gem::Dependency
-  name: database_cleaner
+  name: yard-activesupport-concern
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '2.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.5'
+        version: '2.2'
 - !ruby/object:Gem::Dependency
-  name: yard-activesupport-concern
+  name: rexml
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -101,19 +115,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: appraisal
+  name: webrick
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '1.0'
 description: |-
   ActiveRecord attributes stored serialized in a json column, super smooth.
   For Rails 5.0, 5.1, or 5.2. Typed and cast like Active Record. Supporting nested models,
@@ -153,8 +167,8 @@ files:
 - gemfiles/rails_5_2.gemfile
 - gemfiles/rails_6_0.gemfile
 - gemfiles/rails_6_1.gemfile
+- gemfiles/rails_7_0.gemfile
 - gemfiles/rails_edge.gemfile
-- gemfiles/rails_edge_6.gemfile
 - lib/attr_json.rb
 - lib/attr_json/attribute_definition.rb
 - lib/attr_json/attribute_definition/registry.rb
@@ -182,7 +196,7 @@ licenses:
 metadata:
   homepage_uri: https://github.com/jrochkind/attr_json
   source_code_uri: https://github.com/jrochkind/attr_json
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -197,8 +211,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: ActiveRecord attributes stored serialized in a json column, super smooth.
 test_files: []