perform_later 1.0.6 → 1.0.8

data/lib/perform_later/payload_helper.rb

@@ -3,12 +3,7 @@ module PerformLater
     def self.get_digest(klass, method, *args)
       args = args.flatten
       payload = {:class => klass, :method => method.to_s, :args => args}.to_s
-      
-      puts payload
       digest = Digest::MD5.hexdigest(payload)
-      
-
-      puts "Digest is: #{digest}"
       digest
     end
   end

data/lib/perform_later/version.rb

@@ -1,3 +1,3 @@
 module PerformLater
-  VERSION = "1.0.6"
+  VERSION = "1.0.8"
 end

data/lib/resque/plugins/later/method.rb

@@ -26,12 +26,31 @@ module Resque::Plugins::Later::Method
 
   def perform_later(queue, method, *args)
     ActiveSupport::Deprecation.warn("perform_later will be deprecated in future versions, please use the later method on your models")
+    args = PerformLater::ArgsParser.args_to_resque(args)
+    worker = PerformLater::Workers::ActiveRecord::Worker
+
+    enqueue_in_resque_or_send(worker, queue, method, args)
+  end
+
+  def perform_later!(queue, method, *args)
+    ActiveSupport::Deprecation.warn("perform_later! will be deprecated in future versions, please use the later method on your models")
+
+    args = PerformLater::ArgsParser.args_to_resque(args)
+    digest = PerformLater::PayloadHelper.get_digest(self.class.name, method, args)
+    worker = PerformLater::Workers::ActiveRecord::LoneWorker
     
-    if PerformLater.config.enabled?
-      args = PerformLater::ArgsParser.args_to_resque(args)
-      Resque::Job.create(queue, PerformLater::Workers::ActiveRecord::Worker, self.class.name, self.id, method, *args)
-    else
-      self.send(method, args)
-    end
+    return "AR EXISTS!" unless Resque.redis.get(digest).blank?
+    Resque.redis.set(digest, 'EXISTS')
+
+    enqueue_in_resque_or_send(worker, queue, method, args)
   end
+
+  private 
+    def enqueue_in_resque_or_send(worker, queue, method, args)
+      if PerformLater.config.enabled?
+        Resque::Job.create(queue, worker, self.class.name, self.id, method, *args)   
+      else
+        self.send(method, args)
+      end
+    end
 end

data/spec/lib/perform_later/payload_helper_spec.rb

@@ -9,7 +9,7 @@ describe PerformLater::PayloadHelper do
       user = User.create
 
       digest = Digest::MD5.hexdigest({ :class => "DummyClass", 
-        :method => :some_method, 
+        :method => :some_method.to_s, 
         :args => ["AR:User:#{user.id}"]
         }.to_s)
 
@@ -18,6 +18,4 @@ describe PerformLater::PayloadHelper do
       subject.get_digest("DummyClass", :some_method, args).should == digest
     end
   end
-
-
 end

data/spec/lib/resque/plugins/later/method_spec.rb

@@ -32,6 +32,16 @@ describe Resque::Plugins::Later::Method do
       user.lonely_long_running_method
       Resque.peek(:generic, 0, 20).length.should == 1
     end
+
+    it "should only add a single method to the queue, since the config is with a loner when using perform_later! method" do
+      user = User.create
+      user.perform_later!(:generic, :lonely_long_running_method)
+      user.perform_later!(:generic, :lonely_long_running_method)
+      user.perform_later!(:generic, :lonely_long_running_method)
+      user.perform_later!(:generic, :lonely_long_running_method)
+      user.perform_later!(:generic, :lonely_long_running_method)
+      Resque.peek(:generic, 0, 20).length.should == 1
+    end
   end
 
   context "disabled" do
@@ -48,5 +58,7 @@ describe Resque::Plugins::Later::Method do
     user.should respond_to(:now_long_running_method)
   end
 
+
+
   
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: perform_later
 version: !ruby/object:Gem::Version
-  version: 1.0.6
+  version: 1.0.8
   prerelease: 
 platform: ruby
 authors: