triggerable 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fe45d698af8f0bf86b0d304e31016a41a8fdd89f
-  data.tar.gz: 472fe481cc00e294f9c52663a004635ffd7a4252
+  metadata.gz: 50059bb341314ac2546438f420f9162ec1661715
+  data.tar.gz: 3d523b78004b5820f2a27a08e5736c2efb49abfb
 SHA512:
-  metadata.gz: d23f611c146cb4917f40ba009521071b9bd71993836fe71f797488b03005540eed7824999eeef1c28af23a6dd344d9a3420b528bddc458ca7f253486a07bd173
-  data.tar.gz: d7ca5ca5abfe73fcddb25faf05b010a15c56defeb0f41f2466236991cae2538ea1752a5c075c98cb41b3250128bb53cf0dc6b74a6865812fa6f1cc7b9ccf11f9
+  metadata.gz: ed94209ac03b2fb05b95c4a11e087493c6017c6f015bf3a3e6013516c8c273e453d0ff1250287fc8da95bf4a52347720095e44be15d716e587484fd263b5b76e
+  data.tar.gz: 408aca735a06cdd505ab0280d175b815586245d46df7e4fe6033042fbb3e37bcbc022ba81e6e0b88e5941cfe9fdebf7de900fe47c2b355f07a666924a8cafafc

data/README.md

@@ -66,11 +66,11 @@ trigger on: :after_update, if: -> { orders.any? } do
 end
 ```
 
-If you need to share logic between triggers/automations bodies you can move it into separate class. It should be inherited from `Triggerable::Action` and implement a single method `run_for!(trigger_name, obj)` where trigger_name is a string passed to rule in :name option and obj is a triggered object. Then you can pass a name of your action class instead of do block.
+If you need to share logic between triggers/automations bodies you can move it into separate class. It should be inherited from `Triggerable::Action` and implement a single method `run_for!(obj, trigger_name)` where trigger_name is a string passed to rule in :name option and obj is a triggered object. Then you can pass a name of your action class instead of do block.
 
 ```ruby
 class SendWelcomeSms < Triggerable::Action
-  def run_for! trigger_name, object
+  def run_for! object, trigger_name
     SmsGateway.send_to object.phone, welcome_text
   end
 end

data/lib/triggerable/actions.rb

@@ -21,7 +21,11 @@ module Triggerable
 
     def run_for! object, trigger_name
       proc = @block
-      object.instance_eval { instance_exec(trigger_name, &proc) }
+      object.instance_eval do
+        PaperTrail.whodunnit = trigger_name if defined?(PaperTrail)
+        instance_exec(&proc)
+        PaperTrail.whodunnit = nil if defined?(PaperTrail)
+      end
     end
   end
 end

data/lib/triggerable/version.rb

@@ -1,3 +1,3 @@
 module Triggerable
-  VERSION = '0.1.2'
+  VERSION = '0.1.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: triggerable
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - DmitryTsepelev
@@ -150,7 +150,6 @@ files:
 - spec/integration/conditions_spec.rb
 - spec/integration/short_syntax_spec.rb
 - spec/models.rb
-- spec/names_spec.rb
 - spec/schema.rb
 - spec/scopes_spec.rb
 - spec/spec_helper.rb
@@ -187,7 +186,6 @@ test_files:
 - spec/integration/conditions_spec.rb
 - spec/integration/short_syntax_spec.rb
 - spec/models.rb
-- spec/names_spec.rb
 - spec/schema.rb
 - spec/scopes_spec.rb
 - spec/spec_helper.rb

data/spec/names_spec.rb

@@ -1,19 +0,0 @@
-require 'spec_helper'
-
-describe 'Naming' do
-  before(:each) do
-    Engine.clear
-    TestTask.destroy_all
-  end
-
-  it 'trigger name is available in body' do
-    name = 'Create followup'
-    TestTask.trigger name: name, on: :before_update, if: { status: { is: 'solved' } } do |trigger_name|
-      self.kind = trigger_name
-    end
-
-    task = TestTask.create
-    task.update_attributes status: 'solved'
-    expect(task.kind).to eq(name)
-  end
-end