has_calculated_fields 1.0.3 → 1.0.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a23a9eeb023025474707803883654b56283734136f43a154e808f8791249045
-  data.tar.gz: a9628e8beed6c91953e0eb3573061cb33e5f5fb8f246eedca78851cedf8948d1
+  metadata.gz: 0afedc16cbc98af23216b3d921975247b1852fb83ada078232ae8878605ecd40
+  data.tar.gz: 0c527a880f8dc6164fdfc2328475b6de0327c0bbb172b21940e5ec0f1344f8ec
 SHA512:
-  metadata.gz: aaa4fc3f368d22f30e8b98e1d93744413e2be1d794ac27382500f7c00da2a05c384f235364527058a2332ff4a0908cc987c0b2dbb12d7e98eaeec209623cb51e
-  data.tar.gz: 9b5d7beaeb2718b7b5a2dcbdc58c9ce86501ed1bd91ab140b1b59d01b0682950c5d491a6950e60b03f52ea48ac69b0dc2ac8f87475a6952015d44bf26e9f304d
+  metadata.gz: '081c4f56ed1bd82af9a4fa15a40320d91c57546970681889152f7589832311fb7a6ef582b12587d0143c29cac4f78de4baee31d1a59e6c570408cf996a2c8bfa'
+  data.tar.gz: 1954e0f1faf6b0e7bfe0d7c716d9e8e550745015f8217c0b383498b0d7a1811637053984008b632dd25ed69ecef3c5977fe03ad3219715d1d22c5bc7290bd8cf

data/Gemfile.lock

@@ -1,12 +1,12 @@
 PATH
   remote: .
   specs:
-    has_calculated_fields (1.0.0)
+    has_calculated_fields (1.0.3.4)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    active_model_serializers (0.9.7)
+    active_model_serializers (0.9.8)
       activemodel (>= 3.2)
       concurrent-ruby (~> 1.0)
     activemodel (4.2.11.1)
@@ -21,44 +21,42 @@ GEM
       minitest (~> 5.1)
       thread_safe (~> 0.3, >= 0.3.4)
       tzinfo (~> 1.1)
-    appraisal (2.2.0)
+    appraisal (2.4.1)
       bundler
       rake
       thor (>= 0.14.0)
     arel (6.0.4)
-    ast (2.4.0)
-    builder (3.2.3)
-    byebug (11.0.1)
-    concurrent-ruby (1.1.5)
-    database_cleaner (1.7.0)
-    diff-lcs (1.3)
+    ast (2.4.2)
+    builder (3.2.4)
+    byebug (11.1.3)
+    concurrent-ruby (1.1.10)
+    database_cleaner (1.99.0)
+    diff-lcs (1.5.0)
     docile (1.1.5)
-    factory_bot (4.11.1)
-      activesupport (>= 3.0.0)
     i18n (0.9.5)
       concurrent-ruby (~> 1.0)
-    json (2.2.0)
-    minitest (5.11.3)
-    parallel (1.17.0)
-    parser (2.6.3.0)
-      ast (~> 2.4.0)
-    powerpack (0.1.2)
+    json (2.6.1)
+    minitest (5.15.0)
+    parallel (1.22.1)
+    parser (2.7.2.0)
+      ast (~> 2.4.1)
+    powerpack (0.1.3)
     rainbow (2.2.2)
       rake
-    rake (12.3.3)
-    rspec (3.8.0)
-      rspec-core (~> 3.8.0)
-      rspec-expectations (~> 3.8.0)
-      rspec-mocks (~> 3.8.0)
-    rspec-core (3.8.2)
-      rspec-support (~> 3.8.0)
-    rspec-expectations (3.8.4)
+    rake (13.0.6)
+    rspec (3.11.0)
+      rspec-core (~> 3.11.0)
+      rspec-expectations (~> 3.11.0)
+      rspec-mocks (~> 3.11.0)
+    rspec-core (3.11.0)
+      rspec-support (~> 3.11.0)
+    rspec-expectations (3.11.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-mocks (3.8.1)
+      rspec-support (~> 3.11.0)
+    rspec-mocks (3.11.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.8.0)
-    rspec-support (3.8.2)
+      rspec-support (~> 3.11.0)
+    rspec-support (3.11.0)
     rubocop (0.49.1)
       parallel (~> 1.10)
       parser (>= 2.3.3.1, < 3.0)
@@ -66,18 +64,18 @@ GEM
       rainbow (>= 1.99.1, < 3.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.10.1)
+    ruby-progressbar (1.11.0)
     simplecov (0.15.1)
       docile (~> 1.1.0)
       json (>= 1.8, < 3)
       simplecov-html (~> 0.10.0)
     simplecov-html (0.10.2)
     sqlite3 (1.3.13)
-    thor (0.20.3)
+    thor (1.2.1)
     thread_safe (0.3.6)
-    tzinfo (1.2.5)
+    tzinfo (1.2.9)
       thread_safe (~> 0.1)
-    unicode-display_width (1.6.0)
+    unicode-display_width (1.8.0)
 
 PLATFORMS
   ruby
@@ -88,7 +86,6 @@ DEPENDENCIES
   appraisal (~> 2.2)
   byebug
   database_cleaner
-  factory_bot (~> 4.8)
   has_calculated_fields!
   rspec (~> 3.6)
   rubocop (~> 0.49.1)
@@ -96,4 +93,4 @@ DEPENDENCIES
   sqlite3 (~> 1.3.6)
 
 BUNDLED WITH
-   1.16.6
+   1.17.2

data/has_calculated_fields.gemspec

@@ -3,7 +3,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "has_calculated_fields"
-  s.version     = "1.0.3"
+  s.version     = "1.0.3.4"
   s.platform    = Gem::Platform::RUBY
   s.author      = ["Adrian Fernandez"]
   s.email       = ["adrianfernandez85@gmail.com"]
@@ -24,7 +24,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency "simplecov", "~> 0.15.0"
   s.add_development_dependency "appraisal", "~> 2.2"
   s.add_development_dependency "byebug"
-  s.add_development_dependency "factory_bot", "~> 4.8"
   s.add_development_dependency "database_cleaner"
   s.add_development_dependency "active_model_serializers", "~> 0.9.3"
 end

data/lib/has_calculated_fields/has_calculated_fields.rb

@@ -41,141 +41,172 @@ module HasCalculatedFields
     end
 
     def _has_calculated_fields__on_before_save
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_before_save)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_before_save)
 
       has_calculated_fields_options[:on_before_save].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_save
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_save)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_save)
 
       has_calculated_fields_options[:on_after_save].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_before_validation
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_before_validation)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_before_validation)
 
       has_calculated_fields_options[:on_before_validation].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_validation
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_validation)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_validation)
 
       has_calculated_fields_options[:on_after_validation].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_before_update
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_before_update)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_before_update)
 
       has_calculated_fields_options[:on_before_update].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_before_create
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_before_create)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_before_create)
 
       has_calculated_fields_options[:on_before_create].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_before_destroy
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_before_destroy)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_before_destroy)
 
       has_calculated_fields_options[:on_before_destroy].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_around_save
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_around_save)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_around_save)
 
       has_calculated_fields_options[:on_around_save].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_around_update
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_around_update)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_around_update)
 
       has_calculated_fields_options[:on_around_update].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_around_create
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_around_create)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_around_create)
 
       has_calculated_fields_options[:on_around_create].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_around_destroy
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_around_destroy)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_around_destroy)
 
       has_calculated_fields_options[:on_around_destroy].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_create
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_create)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_create)
 
       has_calculated_fields_options[:on_after_create].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_update
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_update)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_update)
 
       has_calculated_fields_options[:on_after_update].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_destroy
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_destroy)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_destroy)
 
       has_calculated_fields_options[:on_after_destroy].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _has_calculated_fields__on_after_commit
-      return unless respond_to?(:has_calculated_fields_options)
-      return unless has_calculated_fields_options.has_key?(:on_after_commit)
+      return true unless respond_to?(:has_calculated_fields_options)
+      return true unless has_calculated_fields_options.has_key?(:on_after_commit)
 
       has_calculated_fields_options[:on_after_commit].each do |data|
         _process_data(data)
       end
+
+      true
     end
 
     def _process_data(data)
+      # return true unless _should_calculate_data?(data)
       attr_equal = "#{data[:calculated_field]}="
 
       value = case data[:type]
@@ -185,7 +216,6 @@ module HasCalculatedFields
                 _process_method(data)
       end
 
-      puts "#{attr_equal}#{value}"
       send(attr_equal, value)
     end
 
@@ -224,6 +254,16 @@ module HasCalculatedFields
         value
       end
     end
+
+    def _should_calculate_data?(data)
+      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])
+
+      false
+    end
   end
 end
 

data/spec/has_calculated_fields_spec.rb

@@ -5,7 +5,7 @@ describe HasCalculatedFields do
   describe "runs before_save callbacks" do
     let(:name) { "name" }
     let(:created_at) { Time.find_zone("Madrid").parse("2019-05-03 2pm") }
-    let(:obj) { create(:sample_model, created_at: created_at, name: name) }
+    let(:obj) { SampleModel.create(created_at: created_at, name: name) }
 
     context "using time" do
       context "when attribute has value" do
@@ -18,15 +18,6 @@ describe HasCalculatedFields do
             .to("Sat, 04 May 2019 16:00:00 +0200")
         end
       end
-
-      context "when attribute hasn't value" do
-        let(:created_at) { nil }
-        it "assigns formatted time attribute" do
-          expect { obj.save }.to change { obj.calculated_created_at }
-            .from(I18n.l(Time.current))
-            .to(I18n.l(Time.current.in_time_zone("UTC")))
-        end
-      end
     end
 
     context "using method" do
@@ -37,6 +28,25 @@ describe HasCalculatedFields do
           .from("name calculated!")
           .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!")
+      end
+
+      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!")
+      end
     end
   end
 end

data/spec/spec_helper.rb

@@ -3,12 +3,9 @@
 $LOAD_PATH << File.join(File.dirname(__FILE__), "..", "lib")
 require "sqlite3"
 require "has_calculated_fields"
-require "factory_bot"
 require 'database_cleaner'
 require "active_model_serializers"
 
-Dir["./spec/support/**/*.rb"].sort.each { |f| require f }
-
 RSpec.configure do |config|
   config.before(:suite) do
     DatabaseCleaner.strategy = :transaction
@@ -24,13 +21,16 @@ end
 
 ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
 
-ActiveRecord::Schema.define(version: 1) do
+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
 
@@ -48,6 +48,22 @@ class SampleModel < ActiveRecord::Base
       method: Proc.new do |obj|
         obj.name + " calculated!"
       end
+    },
+    {
+      calculated_field: :calculated_conditional_if,
+      type: :method,
+      if_changed: :random_attribute,
+      method: Proc.new do |obj|
+        obj.name + " calculated!"
+      end
+    },
+    {
+      calculated_field: :calculated_conditional_unless,
+      type: :method,
+      unless_changed: :random_attribute,
+      method: Proc.new do |obj|
+        obj.name + " calculated!"
+      end
     }
   ]
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: has_calculated_fields
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.3.4
 platform: ruby
 authors:
 - Adrian Fernandez
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-18 00:00:00.000000000 Z
+date: 2022-05-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -108,20 +108,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: factory_bot
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.8'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '4.8'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -168,15 +154,13 @@ files:
 - lib/has_calculated_fields/.DS_Store
 - lib/has_calculated_fields/has_calculated_fields.rb
 - spec/.DS_Store
-- spec/factories/other_event.rb
 - spec/has_calculated_fields_spec.rb
 - spec/spec_helper.rb
-- spec/support/factory_bot.rb
 homepage: http://github.com/adrian-fernandez/has_calculated_fields
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -192,11 +176,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Rails gem to allow models to save auto calculated fields
 test_files:
-- spec/factories/other_event.rb
 - spec/has_calculated_fields_spec.rb
 - spec/spec_helper.rb
-- spec/support/factory_bot.rb

data/spec/factories/other_event.rb

@@ -1,6 +0,0 @@
-#frozen_string_literal: true
-
-FactoryBot.define do
-  factory :sample_model do
-  end
-end

data/spec/support/factory_bot.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.configure do |config|
-  config.include FactoryBot::Syntax::Methods
-
-  config.before(:suite) do
-    FactoryBot.find_definitions
-  end
-end