postburner 0.6.0 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0c2a65b18596dafa8809d07924e2274f7adc05ce971fea1b82143c7132f2929
-  data.tar.gz: e3000b8fa490aec4fb1c8b49c54fe7ee78613b3e7e3849c2270d7a7dd01adecd
+  metadata.gz: ca9bed4afd8b585d14934b4ba09b20c58109d8de39d5300d135069e2723c3aba
+  data.tar.gz: 5ae5f4a07e53ac69b4d4b795f3b8c8bdf5b9a087e1a8513f6fed03cb47e48210
 SHA512:
-  metadata.gz: 87cb7aec3924346ab43b93f5886aebfb1b138a36b199eb3e10032404a23b07774c287cb96ed8c4889feabde1106aba63d23d6429cead43f3eec8211f257d6baa
-  data.tar.gz: 94e32291c4f00db869a630adeea7e44abb92c4938e2314fc5e46ea487b874acc4a74b80771bd8db8944f3a7c98e06101ef90a36f8da5bf72d7fecc82fbd14c3c
+  metadata.gz: c741884d62b1b71b7c8ce928baff5fef35054c58962f530ee06764b4947477ca394712300256e98409187494092c7164a5f8490482e54a19d0f1776b8465f783
+  data.tar.gz: 193a2e625139d0a441503afc15d64888eb25234d0ae81ca320a319506130e8eb1c960141d60b97066633f6a777ab47fe21d49d42bc448497b4225e8c78924051

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## v0.6.1 - 2021-11-01
+
+### Updated
+- update run_at logic to support only inserting after save commit.
+
 ## v0.6.0 - 2021-11-01
 
 ### Added

data/app/models/postburner/job.rb

@@ -24,25 +24,31 @@ module Postburner
 
     before_validation :ensure_sid!
     before_destroy :delete!
+    after_save_commit :insert_if_queued!
 
     validates :sid, presence: {strict: true}
 
     def queue!(options={})
       return if self.queued_at.present? && self.bkid.present?
+      raise ActiveRecord::RecordInvalid, "Can't queue unless valid." unless self.valid?
+      at = options.delete(:at)
 
-      self.save!
+      self.run_at = case
+                    when at.present?
+                      # this is rudimentary, add error handling
+                      options[:delay] ||= at.to_i - Time.zone.now.to_i
+                      at
+                    when options[:delay].present?
+                      Time.zone.now + options[:delay].seconds
+                    end
 
-      case
-      when options[:at].present?
-        # this is rudimentary, add error handling
-        options[:delay] ||= options[:at].to_i - Time.zone.now.to_i
-        update_column :run_at, options[:at]
-        options.delete(:at)
-      when options[:delay].present?
-        update_column :run_at, Time.zone.now + options[:delay].seconds
-      end
+      @_insert_options = options
+
+      self.save!
+    end
 
-      insert!(options)
+    def will_insert?
+      @_insert_options.is_a? Hash
     end
 
     # tube: backburner.worker.queue.backburner-jobs
@@ -224,6 +230,10 @@ module Postburner
       }.merge(data))
     end
 
+    def insert_if_queued!
+      return unless self.will_insert?
+      insert!(@_insert_options)
+    end
 
     def insert!(options={})
       response = Backburner::Worker.enqueue(

data/lib/postburner/version.rb

@@ -1,3 +1,3 @@
 module Postburner
-  VERSION = '0.6.0'
+  VERSION = '0.6.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: postburner
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors:
 - Matt Smith