active_data 1.1.7 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5de412d5036c1e3192c87b9c7c7cc553cccaa5d53dc48adb32d992c3cf9fca25
-  data.tar.gz: 52c67fb161d13a2f7b4791f048c662962b2f0355b77fb8f10d434e6e5279692f
+  metadata.gz: 80daeb6d630cc5fc243193c917e0b43f19b2fce35a746200551921f3892e1677
+  data.tar.gz: 4fc2d73f4d3a59ce0804fa652ba782ee33997e9be4870b4ee402d67b920a8c45
 SHA512:
-  metadata.gz: a804cb81e49839e78dbd9de0bd7c670bfde5b34c46651009f15bc70b890edae3a2148d0fd48251179fe83068769c6ddb950969cb2f8eb06b116e1c184d2724e2
-  data.tar.gz: 6b8277845088785bb3a4cd95eb2c56882a7ece7c1cb395e04106717a1ec7ace440709b70c7bec9150be2c9b48626cbb458e8c6882ab01ac808067e428f1a55c3
+  metadata.gz: 3d8fe25346c0e72ac3600ed8f89266864cf86081d8a0db463e4b856688e7c5cd7a936b76c872f7470a0f942989ecc51bf5e1a0615ea68f74855119910607b139
+  data.tar.gz: c0f31ed6be42bbe6e3d06f82e4d546c04b72b7a527cb628019516531b430273a89eae8c727a4c4edc2010aa3cdd80fa8195953f793296e4726b2d5f666b973ec

data/.github/workflows/ci.yml

@@ -0,0 +1,35 @@
+name: CI
+on: [push, pull_request]
+jobs:
+  rspec:
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+          - { ruby: '2.3', rails: '4.2' }
+          - { ruby: '2.4', rails: '5.0' }
+          - { ruby: '2.5', rails: '5.1' }
+          - { ruby: '2.6', rails: '5.2' }
+          - { ruby: '2.7', rails: '6.0' }
+          - { ruby: '3.0', rails: '6.1' }
+          - { ruby: '3.0', rails: '7.0' }
+    runs-on: ubuntu-latest
+    env:
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/rails.${{ matrix.rails }}.gemfile
+    steps:
+    - uses: actions/checkout@v2
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+    - run: bundle exec rspec
+
+  rubocop:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 3.0
+        bundler-cache: true
+    - run: bundle exec rubocop

data/Appraisals

@@ -1,4 +1,4 @@
-%w[4.2 5.1 5.2 6.0].each do |version|
+%w[4.2 5.0 5.1 5.2 6.0 6.1 7.0].each do |version|
   appraise "rails.#{version}" do
     gem 'activesupport', "~> #{version}.0"
     gem 'activemodel', "~> #{version}.0"

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # master
 
+# Version 1.2.0
+
+* Rails 6.1 and 7 support (#80). Thanks to @ojab and @rewritten
+
 # Version 1.1.7
 
 * Add typecasting from `ActionController::Parameters` to `Hash` (#73)

data/gemfiles/rails.5.0.gemfile

@@ -0,0 +1,15 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activesupport", "~> 5.0.0"
+gem "activemodel", "~> 5.0.0"
+gem "activerecord", "~> 5.0.0"
+gem "sqlite3", "~> 1.3.6"
+
+group :test do
+  gem "guard"
+  gem "guard-rspec"
+end
+
+gemspec path: "../"

data/gemfiles/rails.6.1.gemfile

@@ -0,0 +1,14 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activesupport", "~> 6.1.0"
+gem "activemodel", "~> 6.1.0"
+gem "activerecord", "~> 6.1.0"
+
+group :test do
+  gem "guard"
+  gem "guard-rspec"
+end
+
+gemspec path: "../"

data/gemfiles/rails.7.0.gemfile

@@ -0,0 +1,14 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activesupport", "~> 7.0.0"
+gem "activemodel", "~> 7.0.0"
+gem "activerecord", "~> 7.0.0"
+
+group :test do
+  gem "guard"
+  gem "guard-rspec"
+end
+
+gemspec path: "../"

data/lib/active_data/model/representation.rb

@@ -1,3 +1,4 @@
+require 'active_model/version'
 require 'active_data/model/attributes/reflections/represents'
 require 'active_data/model/attributes/represents'
 
@@ -67,14 +68,24 @@ module ActiveData
       #
       def emerge_represented_attributes_errors!
         self.class.represented_attributes.each do |attribute|
-          key = :"#{attribute.reference}.#{attribute.column}"
-          # Rails 5 pollutes messages with an empty array on key data fetch attempt
-          messages = errors.messages[key] if errors.messages.key?(key)
-          if messages.present?
-            errors[attribute.column].concat(messages)
-            errors.delete(key)
+          move_errors(:"#{attribute.reference}.#{attribute.column}", attribute.column)
+        end
+      end
+
+      if ActiveModel.version >= Gem::Version.new('6.1.0')
+        def move_errors(from, to)
+          errors[from].each do |error_message|
+            errors.add(to, error_message)
+            errors.delete(from)
           end
         end
+      else # up to 6.0.x
+        def move_errors(from, to)
+          return unless errors.messages.key?(from) && errors.messages[from].present?
+
+          errors[to].concat(errors.messages[from])
+          errors.delete(from)
+        end
       end
     end
   end

data/lib/active_data/model/scopes.rb

@@ -37,16 +37,44 @@ module ActiveData
           super || self.class._scope_model.respond_to?(method)
         end
 
-        def method_missing(method, *args, &block)
-          with_scope do
-            model = self.class._scope_model
-            if model.respond_to?(method)
-              self.class._scope_model.public_send(method, *args, &block)
-            else
-              super
+        # rubocop:disable Style/MethodMissing
+        # rubocop-0.52.1 doesn't understand that `#respond_to_missing?` is defined above
+        if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('3.0.0')
+          def method_missing(method, *args, **kwargs, &block)
+            with_scope do
+              model = self.class._scope_model
+              if model.respond_to?(method)
+                result = model.public_send(method, *args, **kwargs, &block)
+                result.is_a?(ActiveData::Model::Scopes) ? result : model.scope_class.new(result)
+              else
+                super
+              end
+            end
+          end
+        elsif Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.7.0')
+          def method_missing(method, *args, **kwargs, &block)
+            with_scope do
+              model = self.class._scope_model
+              if model.respond_to?(method)
+                model.public_send(method, *args, **kwargs, &block)
+              else
+                super
+              end
+            end
+          end
+        else # up to 2.6.x
+          def method_missing(method, *args, &block)
+            with_scope do
+              model = self.class._scope_model
+              if model.respond_to?(method)
+                model.public_send(method, *args, &block)
+              else
+                super
+              end
             end
           end
         end
+        # rubocop:enable Style/MethodMissing
 
         def with_scope
           previous_scope = self.class._scope_model.current_scope

data/lib/active_data/model/validations/associated.rb

@@ -6,7 +6,7 @@ module ActiveData
           invalid_records = Array.wrap(value).reject do |r|
             r.respond_to?(:valid?) && r.valid?(record.validation_context)
           end
-          record.errors.add(attribute, :invalid, options.merge(value: value)) if invalid_records.present?
+          record.errors.add(attribute, :invalid, **options.merge(value: value)) if invalid_records.present?
         end
       end
 

data/lib/active_data/model/validations/nested.rb

@@ -1,3 +1,5 @@
+require 'active_model/version'
+
 module ActiveData
   module Model
     module Validations
@@ -5,19 +7,26 @@ module ActiveData
         def self.validate_nested(record, name, value)
           if value.is_a?(Enumerable)
             value.each.with_index do |object, i|
-              if yield(object)
-                object.errors.each do |key, message|
-                  key = "#{name}.#{i}.#{key}"
-                  record.errors[key] << message
-                  record.errors[key].uniq!
-                end
-              end
+              import_errors(object.errors, record.errors, "#{name}.#{i}") if yield object
+            end
+          elsif value
+            import_errors(value.errors, record.errors, name.to_s) if yield value
+          end
+        end
+
+        if ActiveModel.version >= Gem::Version.new('6.1.0')
+          def self.import_errors(from, to, prefix)
+            from.each do |error|
+              key = "#{prefix}.#{error.attribute}"
+              to.import(error, attribute: key) unless to.added?(key, error.type, error.options)
             end
-          elsif value && yield(value)
-            value.errors.each do |key, message|
-              key = "#{name}.#{key}"
-              record.errors[key] << message
-              record.errors[key].uniq!
+          end
+        else # up to 6.0.x
+          def self.import_errors(from, to, prefix)
+            from.each do |key, message|
+              key = "#{prefix}.#{key}"
+              to[key] << message
+              to[key].uniq!
             end
           end
         end

data/lib/active_data/version.rb

@@ -1,3 +1,3 @@
 module ActiveData
-  VERSION = '1.1.7'.freeze
+  VERSION = '1.2.0'.freeze
 end

data/spec/lib/active_data/model/attributes/localized_spec.rb

@@ -93,7 +93,7 @@ describe ActiveData::Model::Attributes::Localized do
         before do
           require 'i18n/backend/fallbacks'
           I18n::Backend::Simple.send(:include, I18n::Backend::Fallbacks)
-          I18n.fallbacks.map(en: :ru)
+          I18n.fallbacks[:en] = [:ru]
         end
         after { I18n.fallbacks = false }
         its(:name) { should == 'Привет' }

data/spec/lib/active_data/model/typecasting_spec.rb

@@ -66,11 +66,11 @@ describe ActiveData::Model::Attributes do
 
     specify { expect(model.new(column: 'hello').column).to be_nil }
     specify { expect(model.new(column: '123hello').column).to be_nil }
-    specify { expect(model.new(column: '123').column).to eq(BigDecimal.new('123.0')) }
+    specify { expect(model.new(column: '123').column).to eq(BigDecimal('123.0')) }
     specify { expect(model.new(column: '123.').column).to be_nil }
-    specify { expect(model.new(column: '123.5').column).to eq(BigDecimal.new('123.5')) }
-    specify { expect(model.new(column: 123).column).to eq(BigDecimal.new('123.0')) }
-    specify { expect(model.new(column: 123.5).column).to eq(BigDecimal.new('123.5')) }
+    specify { expect(model.new(column: '123.5').column).to eq(BigDecimal('123.5')) }
+    specify { expect(model.new(column: 123).column).to eq(BigDecimal('123.0')) }
+    specify { expect(model.new(column: 123.5).column).to eq(BigDecimal('123.5')) }
     specify { expect(model.new(column: nil).column).to be_nil }
     specify { expect(model.new(column: [123.5]).column).to be_nil }
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_data
 version: !ruby/object:Gem::Version
-  version: 1.1.7
+  version: 1.2.0
 platform: ruby
 authors:
 - pyromaniac
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-20 00:00:00.000000000 Z
+date: 2021-12-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -200,11 +200,11 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".codeclimate.yml"
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
 - Appraisals
 - CHANGELOG.md
 - Gemfile
@@ -214,9 +214,12 @@ files:
 - Rakefile
 - active_data.gemspec
 - gemfiles/rails.4.2.gemfile
+- gemfiles/rails.5.0.gemfile
 - gemfiles/rails.5.1.gemfile
 - gemfiles/rails.5.2.gemfile
 - gemfiles/rails.6.0.gemfile
+- gemfiles/rails.6.1.gemfile
+- gemfiles/rails.7.0.gemfile
 - lib/active_data.rb
 - lib/active_data/active_record/associations.rb
 - lib/active_data/active_record/nested_attributes.rb
@@ -345,7 +348,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: Working with hashes in AR style

data/.travis.yml

@@ -1,16 +0,0 @@
-sudo: false
-
-matrix:
-  include:
-    - rvm: 2.3.8
-      gemfile: gemfiles/rails.4.2.gemfile
-    - rvm: 2.4.7
-      gemfile: gemfiles/rails.5.1.gemfile
-    - rvm: 2.5.6
-      gemfile: gemfiles/rails.5.2.gemfile
-    - rvm: 2.6.4
-      gemfile: gemfiles/rails.6.0.gemfile
-
-script:
-  - bundle exec rspec
-  - bundle exec rubocop