active_record_compose 0.1.4 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f0306635ed35d2888c6a8f5ddc3d0ac5b04ba6aee2a2f6eaba304e57bc05ebc
-  data.tar.gz: 3fdd952bc0c2b93a448f2be8b8d9a441afb20d635eab9a32ed05b6a837978d7e
+  metadata.gz: dd2ae31a400b17c7ab14c90bd1440060676590c23f91a61f7f54e9a7decf0294
+  data.tar.gz: 4a599ee2da8a5dcc3b0d9279a360b611b2e38f72304e8a837271f01e2b9821b2
 SHA512:
-  metadata.gz: cd5b513cfe39716ea5caf87be10e5fa9e514d134bed93b54456ce8bdf071a9ec5d336efbef8f609c4d3857e3763e4ed75b757db3145ca3cf4c7e79b6c98d6826
-  data.tar.gz: 5e92989ed498e4e244692031231ad2f3c738025a1d066f2146f4ac36b37107143c46d131a30d7fb84df22ac703d230c9062446b5049a60012dd3209fdabe5e88
+  metadata.gz: 18f3632810ffc48ed4bd10f78c4e54ff3c383d34231289f9639e488f270bbfd623a5f002b9f4afc653762611621c6f96b85b52498c2ba34e942d0c7c7288bfe4
+  data.tar.gz: 45fc861a9d223da11b841800dbc0ab689160cabdfdfda3a69433cb1d98884f76f30c742319ffc1ad3601527e9464b322456e44b73c6f098355f898909b5e340c

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [0.1.6] - 2024-01-14
+
+- add doc for `#save` and `#save!`.
+- implement `#save` for symmetry with `#save!`
+- add `InnerModel#initialize` doc.
+
+## [0.1.5] - 2024-01-11
+
+- when invalid, raises ActiveRecord::RecordInvalid on #save!
+
 ## [0.1.4] - 2024-01-10
 
 - remove uniquely defined exception class.

data/lib/active_record_compose/inner_model.rb

@@ -6,6 +6,7 @@ module ActiveRecordCompose
   class InnerModel
     # @param model [Object] the model instance.
     # @param context [Symbol] :save or :destroy
+    # @param context [Proc] proc returning either :save or :destroy
     def initialize(model, context: :save)
       @model = model
       @context = context
@@ -19,6 +20,22 @@ module ActiveRecordCompose
       ret.presence_in(%i[save destroy]) || :save
     end
 
+    # Execute save or destroy. Returns true on success, false on failure.
+    # Whether save or destroy is executed depends on the value of context.
+    #
+    # @return [Boolean] returns true on success, false on failure.
+    def save
+      case context
+      when :destroy
+        model.destroy
+      else
+        model.save
+      end
+    end
+
+    # Execute save or destroy. Unlike #save, an exception is raises on failure.
+    # Whether save or destroy is executed depends on the value of context.
+    #
     # @return [InnerModel] self
     def save!
       case context

data/lib/active_record_compose/model.rb

@@ -22,13 +22,28 @@ module ActiveRecordCompose
       super(attributes)
     end
 
+    # Save the models that exist in models.
+    # Returns false if any of the targets fail, true if all succeed.
+    #
+    # The save is performed within a single transaction.
+    #
+    # @return [Boolean] returns true on success, false on failure.
     def save
       return false if invalid?
 
       save_in_transaction { run_callbacks(:save) { save_models } }
     end
 
-    def save! = save || raise_on_save_error
+    # Save the models that exist in models.
+    # Unlike #save, an exception is raises on failure.
+    #
+    # Saving, like `#save`, is performed within a single transaction.
+    #
+    def save!
+      valid? || raise_validation_error
+
+      save_in_transaction { run_callbacks(:save) { save_models } } || raise_on_save_error
+    end
 
     # Behavior is same to `#save`, but `before_create` and `after_create` hooks fires.
     #
@@ -65,7 +80,10 @@ module ActiveRecordCompose
     # Behavior is same to `#create`, but raises an exception prematurely on failure.
     #
     def create!(attributes = {})
-      create(attributes) || raise_on_save_error
+      assign_attributes(attributes)
+      valid? || raise_validation_error
+
+      save_in_transaction { run_callbacks(:save) { run_callbacks(:create) { save_models } } } || raise_on_save_error
     end
 
     # Behavior is same to `#save`, but `before_update` and `after_update` hooks fires.
@@ -103,7 +121,10 @@ module ActiveRecordCompose
     # Behavior is same to `#update`, but raises an exception prematurely on failure.
     #
     def update!(attributes = {})
-      update(attributes) || raise_on_save_error
+      assign_attributes(attributes)
+      valid? || raise_validation_error
+
+      save_in_transaction { run_callbacks(:save) { run_callbacks(:update) { save_models } } } || raise_on_save_error
     end
 
     private
@@ -130,6 +151,8 @@ module ActiveRecordCompose
 
     def save_models = wrapped_models.all? { _1.save! }
 
+    def raise_validation_error = raise ActiveRecord::RecordInvalid, self
+
     def raise_on_save_error = raise ActiveRecord::RecordNotSaved.new('Failed to save the model.', self)
   end
 end

data/lib/active_record_compose/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecordCompose
-  VERSION = '0.1.4'
+  VERSION = '0.1.6'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_record_compose
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.6
 platform: ruby
 authors:
 - hamajyotan
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-09 00:00:00.000000000 Z
+date: 2024-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -65,7 +65,7 @@ metadata:
   homepage_uri: https://github.com/hamajyotan/active_record_compose
   source_code_uri: https://github.com/hamajyotan/active_record_compose
   changelog_uri: https://github.com/hamajyotan/active_record_compose/blob/main/CHANGELOG.md
-  documentation_uri: https://www.rubydoc.info/gems/active_record_compose/0.1.4
+  documentation_uri: https://www.rubydoc.info/gems/active_record_compose/0.1.6
   rubygems_mfa_required: 'true'
 post_install_message:
 rdoc_options: []