has_calculated_fields 1.0.3.6 → 1.0.3.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a19bc37b89db9ad306518d885b96897f2bbcda5471a3035ae04c5ee7ba30429
-  data.tar.gz: 6044d0bc9cff905e0c48e123595f387f601d32f859719bae47655fa45b18866e
+  metadata.gz: f6cf158f78752207c08624fc9f0a78e61fe23bce35345390c68199f8a60ed38e
+  data.tar.gz: c5f79aacc2c41741e52f9916cfd5b7aef7196c9088d13a6c70ba122d1adcb24e
 SHA512:
-  metadata.gz: 7872e4d93d3febee83c3245108d14dda74dcab6bac04d1a450c0013dbfaca7e38b8b0d3ddd67c7a2deaf8eec7c4203977b68bc2508182ddfd25d9d443a9b73f4
-  data.tar.gz: 90ae2e5dd0ded0c0248c90f5d02e7f8a91fbfc7c7c5041ddeadfb4483460224d2ec363e50b6877cc0494d386491cf0edb1c48ebe33a3fc7dd56fe9ae500c7ef3
+  metadata.gz: 7c2ccaefb0403244e27280e269078f5356024092af4c71c3d3e08b191ec583ce0717558c6cd3200c89c6d1ff505714ea560bcf1f2c7c09a634c879f210505ad4
+  data.tar.gz: 11e1c6edd54e3d9c8d6ff9660eaaa52d660e67ea4aecb40428639936acd70b70acd3873daed4211d3abc82bb9ddf822951913a3f871c052478c0ad1c557b6117

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    has_calculated_fields (1.0.3.4)
+    has_calculated_fields (1.0.3.7)
 
 GEM
   remote: https://rubygems.org/

data/has_calculated_fields.gemspec

@@ -3,7 +3,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "has_calculated_fields"
-  s.version     = "1.0.3.6"
+  s.version     = "1.0.3.7"
   s.platform    = Gem::Platform::RUBY
   s.author      = ["Adrian Fernandez"]
   s.email       = ["adrianfernandez85@gmail.com"]

data/lib/has_calculated_fields/has_calculated_fields.rb

@@ -206,7 +206,8 @@ module HasCalculatedFields
     end
 
     def _process_data(data)
-      # return true unless _should_calculate_data?(data)
+      return true unless _should_calculate_data?(data)
+
       attr_equal = "#{data[:calculated_field]}="
 
       value = case data[:type]
@@ -255,12 +256,17 @@ module HasCalculatedFields
       end
     end
 
-    def _should_calculate_data?(data)
+    def _should_calculate_data?(data = {})
+      if_changed = data.fetch(:if_changed, nil)
+      unless_changed = data.fetch(:unless_changed, nil)
+
+      changed_keys = changes.keys.map(&:to_sym) || []
+
       return true if data.blank?
       return true if !data.has_key?(:if_changed) && !data.has_key?(:unless_changed)
 
-      return true if data.has_key?(:if_changed) && changes.keys.map(&:to_sym).include?(data[:if_changed])
-      return true if data.has_key?(:unless_changed) && !changes.keys.map(&:to_sym).include?(data[:unless_changed])
+      return true if data.has_key?(:if_changed) && changed_keys.include?(if_changed)
+      return true if data.has_key?(:unless_changed) && !changed_keys.include?(unless_changed)
 
       false
     end

data/lib/has_calculated_fields.rb

@@ -2,7 +2,6 @@
 
 require "active_record"
 require "active_support/inflector"
-require "active_model_serializers"
 
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 

data/spec/has_calculated_fields_spec.rb

@@ -29,23 +29,42 @@ describe HasCalculatedFields do
           .to("new name calculated!")
       end
 
-      it "assigns conditional when condition is matched" do
-        obj.name = "conditional name"
-        obj.random_attribute = "4"
-
-        expect { obj.save }.to change {
-          obj.calculated_conditional_unless
-        }.and change { obj.calculated_conditional_if }
-          .to("conditional name calculated!")
+      context "if_changed" do
+        it "assigns conditional when condition is matched" do
+          obj.name = "conditional name"
+          obj.random_attribute = "4"
+
+          expect { obj.save }.to change {
+            obj.calculated_conditional_if
+          }
+        end
+
+        it "does not assign conditional when condition is matched" do
+          obj.name = "conditional name"
+          obj.random_attribute = "4"
+
+          expect { obj.save }.not_to change {
+            obj.calculated_conditional_if
+          }
+        end
       end
 
-      it "assigns conditional when condition is matched" do
-        obj.name = "conditional name"
+      context "unless_changed" do
+        it "assigns conditional when condition is matched" do
+          obj.name = "conditional name"
 
-        expect { obj.save }.to change {
-          obj.calculated_conditional_if
-        }.and change { obj.calculated_conditional_unless }
-          .to("conditional name calculated!")
+          expect { obj.save }.to change {
+            obj.calculated_conditional_unless
+          }
+        end
+
+        it "does not assign conditional when condition is matched" do
+          obj.name = "conditional name"
+
+          expect { obj.save }.not_to change {
+            obj.calculated_conditional_unless
+          }
+        end
       end
     end
   end

data/spec/schema.rb

@@ -0,0 +1,12 @@
+ActiveRecord::Schema.define(version: 2022_05_07_120518) do
+  create_table :sample_models do |t|
+    t.string :name
+    t.string :random_attribute
+    t.datetime :created_at
+
+    t.string :calculated_name
+    t.string :calculated_created_at
+    t.string :calculated_conditional_if
+    t.string :calculated_conditional_unless
+  end
+end

data/spec/spec_helper.rb

@@ -4,7 +4,6 @@ $LOAD_PATH << File.join(File.dirname(__FILE__), "..", "lib")
 require "sqlite3"
 require "has_calculated_fields"
 require 'database_cleaner'
-require "active_model_serializers"
 
 RSpec.configure do |config|
   config.before(:suite) do
@@ -19,20 +18,9 @@ RSpec.configure do |config|
   end
 end
 
-ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
-
-ActiveRecord::Schema.define(version: 3) do
-  create_table :sample_models do |t|
-    t.string :name
-    t.string :random_attribute
-    t.datetime :created_at
-
-    t.string :calculated_name
-    t.string :calculated_created_at
-    t.string :calculated_conditional_if
-    t.string :calculated_conditional_unless
-  end
-end
+ActiveRecord::Base.establish_connection(adapter: 'sqlite3', database: ':memory:')
+ActiveRecord::Schema.verbose = true
+load "spec/schema.rb"
 
 class SampleModel < ActiveRecord::Base
   has_calculated_fields on_before_save: [

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: has_calculated_fields
 version: !ruby/object:Gem::Version
-  version: 1.0.3.6
+  version: 1.0.3.7
 platform: ruby
 authors:
 - Adrian Fernandez
@@ -155,6 +155,7 @@ files:
 - lib/has_calculated_fields/has_calculated_fields.rb
 - spec/.DS_Store
 - spec/has_calculated_fields_spec.rb
+- spec/schema.rb
 - spec/spec_helper.rb
 homepage: http://github.com/adrian-fernandez/has_calculated_fields
 licenses:
@@ -181,4 +182,5 @@ specification_version: 4
 summary: Rails gem to allow models to save auto calculated fields
 test_files:
 - spec/has_calculated_fields_spec.rb
+- spec/schema.rb
 - spec/spec_helper.rb