purgatory 3.1.1 → 3.1.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YWJlNzE4NGQyODkzYzM5NzRlZWY3NjUzMTRkODNkNDQ2Y2ExZTVlNQ==
+    YmZmZDQ1M2NjYTQ3YTU4MGY1YjdjNWY4ZjNlNGVjM2QyYjIxNDE4Mw==
   data.tar.gz: !binary |-
-    M2NmMmQ4ZTY2MDIxZTBlOTczMTgwZTM0ODA0ZmMwODAwNjUwMTRhMw==
+    YmI2MzNkNmUyZmVjNGVhNTJiYTEzMTMwM2YyOWU3ZmNkNzg0NWMyYw==
 SHA512:
   metadata.gz: !binary |-
-    ZTkyMGYyYTE4ZDE4OGZjMTE4OTc2ZDFhYjIzYTA4ZTI5MmI4YzlkMzhjMjNm
-    MTQ2MWQ2YzlkZGIxZmJmMGM1OGIzNmVhNTYyYTZiNTI4ZGRlOWFjNGY1Y2Rm
-    ZjE2Y2RlMjIyYzY0Y2MwMjE4ZWNmZGY0NTU4MGIxOTVlYTUyOTk=
+    NTEyYTBhMTBkMzc1ODhmYTNjOTFmMjkzMDMwOWI0MzZiZDcxM2JhOTFiNGM2
+    MGRhZmY3ZWY1MjUyNTdjYTY2MWNlZTkwZjJiMmNiNGFhNDIyNjVjM2NmYTlj
+    MWMxM2Q2ZmZmM2U4OTRlZmNkYjdhMWZjOTRlNTk4MmYyNmE5NjU=
   data.tar.gz: !binary |-
-    ZDI2OGZlOTRkYTdjYWZmMDcxODNjYjE2MjYwYjg1ZmMwMmQ3NGVkYmE1MGEw
-    NGIwZTdiNjliZTA2YjZhNzg5MTcxZjY4YWUzNzVkZWU3ODBkZGJjNmE1NGNk
-    MTI3OGIwZDMzN2RlYWNmMTUxN2M0YjFiYjIwY2RkNzJmMmUwNTI=
+    ZTA0NDY5NDliOTFjMTcyMzcyZGRkOGViZGI5ZWUwZTVkNzM4MjliZjUxMzU2
+    NTNjYTMwMDM0MmRjODZjYzdiNzc0YmVlYzMwYzg2MDY4YTMyZmRkMDViMDBk
+    ZDYzNmFiMjBkMTg4MjU1MDMzNzhlNzFiNmZiNTg5MTViMjdiMWM=

data/VERSION

@@ -1 +1,2 @@
-3.1.1
+3.1.2
+

data/lib/purgatory/purgatory.rb

@@ -44,20 +44,22 @@ class Purgatory < ActiveRecord::Base
   end
 
   def approve!(approver = nil)
-    return false if approved?
-    success = soul_with_changes.save
-    if performable_method.present? && success
-      success = soul.send(performable_method[:method],*performable_method[:args])
-    end
+    self.with_lock do
+      return false if approved?
+      success = soul_with_changes.save
+      if performable_method.present? && success
+        success = soul.send(performable_method[:method],*performable_method[:args])
+      end
 
-    if success
-      self.approver = approver
-      self.approved_at = Time.now
-      self.soul_id = soul.id
-      save
-      return true
+      if success
+        self.approver = approver
+        self.approved_at = Time.now
+        self.soul_id = soul.id
+        save
+        return true
+      end
+      false
     end
-    false
   end
 
   def self.pending_with_matching_soul(soul)

data/purgatory.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: purgatory 3.1.1 ruby lib
+# stub: purgatory 3.1.2 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "purgatory"
-  s.version = "3.1.1"
+  s.version = "3.1.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Elan Dubrofsky"]
-  s.date = "2017-08-21"
+  s.date = "2017-08-24"
   s.description = "Put your model changes in purgatory and allow them to remain lost souls until they are approved"
   s.email = "elan.dubrofsky@gmail.com"
   s.extra_rdoc_files = [

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: purgatory
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 3.1.2
 platform: ruby
 authors:
 - Elan Dubrofsky
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-21 00:00:00.000000000 Z
+date: 2017-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rdoc