facio 0.1.13 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78fd84c0df628d66cf2ae1ee9776fbdce89015c8d0451723662f60caa2893995
-  data.tar.gz: 610a113b4434bc52a215b707930fc6accb9a0cbe7cdc1a4e2b4b3bbdaef616cd
+  metadata.gz: b6480a8cab348b66eb9f4987d4c56ee773faf649d9cfd937434a6afd3f822ac3
+  data.tar.gz: 4f2bc2289286be83807973409daee4f845faca070abaae9f383326b38cef5f4e
 SHA512:
-  metadata.gz: 218f4858fb6583b7a86efcaec1f6fa37010c0829c431df05371d69ed433917905a5c7b42ab58b8893fdb2795006f73325392104181d40e3e57441ae316d3d6a7
-  data.tar.gz: 42576dbb7026b7b034ac80334090386d0a40763b05b7eaa26e765efb389120e59b450e759b3c548271f395fa632ab7bb51bd31feca9771fa282f77e6a27674de
+  metadata.gz: b685cee0737c95408dd2a5199e76f510f525339464dc104a5d97706c5d0309222b5bc2bb205cea818be77ec2358deb817d54bafa26e4ba389840f020f164c7e3
+  data.tar.gz: 544bff591787982d6249968921a919a3396cc13d8c56197b94aaceb32a2a9f7ebc656b5530db112dd7ef6dc42c602502fb8992b465f1de6bbe3149c3d274b5e2

data/lib/facio/concerns/execution.rb

@@ -1,8 +1,32 @@
 module Execution
   extend ActiveSupport::Concern
 
+  # Facio doesn't support arguments in the perform method, that's done with the context object.
+  # This is why we override perform here.
   def facio_perform_with_arguments(*args)
-    facio_perform_without_arguments
+    @context = self.class.context_class.new(arguments.first)
+    @result = self.class.result_class&.new
+    @performed = false
+
+    if @context.valid?
+      result = nil
+      if transactional && defined?(ActiveRecord::Base)
+        ActiveRecord::Base.transaction(requires_new: true) do
+          # result = block.call
+          result = facio_perform_without_arguments
+          # This will only rollback the changes of the service, SILENTLY, however the context will be failed? already.
+          # This is the most close to expected behaviour this can get.
+          raise ActiveRecord::Rollback if context.failed?
+        end
+
+      else
+        result = facio_perform_without_arguments
+      end
+
+      @performed = true
+      # Purely as a convenience, but also to enforce a standard
+      context.result ||= result if @result.nil?
+    end
   end
 
   included do

data/lib/facio/service.rb

@@ -11,7 +11,6 @@ module Facio
     include ServiceResult
     include Transactional
     include Translations
-    include Callbacks
     include Execution
 
     class << self
@@ -21,6 +20,20 @@ module Facio
         job
       end
 
+      def perform_later(...)
+        job = job_or_instantiate(...)
+
+        enqueue_result = job.enqueue
+
+        job.instance_variable_set(:@performed, false)
+        job.instance_variable_set(:@context, context_class.new(job.arguments.first))
+        job.instance_variable_set(:@result, result_class&.new)
+
+        yield job if block_given?
+
+        enqueue_result
+      end
+
       alias_method :perform_now, :perform
     end
 

data/lib/facio/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Facio
-  VERSION = "0.1.13"
+  VERSION = "0.1.15"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: facio
 version: !ruby/object:Gem::Version
-  version: 0.1.13
+  version: 0.1.15
 platform: ruby
 authors:
 - Tom de Grunt