redpear 0.7.3 → 0.7.4

data/lib/redpear/model.rb

@@ -96,7 +96,7 @@ class Redpear::Model < Hash
     super()
     store 'id', (attrs.delete("id") || attrs.delete(:id) || self.class.pk_counter.next).to_s
     update(attrs)
-    self.class.members.add(id)
+    after_create(attrs)
   end
 
   # @return [String] the ID of this record
@@ -179,14 +179,12 @@ class Redpear::Model < Hash
   # @return [Hash]
   def update(hash)
     clear
-    self.class.transaction do
-      bulk = {}
-      hash.each do |name, value|
-        column = self.class.columns[name] || next
-        store_attribute bulk, column, value
-      end
-      attributes.merge! bulk
+    bulk = {}
+    hash.each do |name, value|
+      column = self.class.columns[name] || next
+      store_attribute bulk, column, value
     end
+    attributes.merge! bulk
     self
   end
 
@@ -214,10 +212,11 @@ class Redpear::Model < Hash
   # Destroy the record.
   # @return [Boolean] true if successful
   def destroy
-    lookups # Build before transaction
+    before_destroy
     self.class.transaction do
       lookups.each {|l| l.delete(id) }
       attributes.purge!
+      after_destroy
     end
     freeze
   end
@@ -249,7 +248,27 @@ class Redpear::Model < Hash
 
     value
   end
-  protected :store, :store_attribute
+
+  # Cheap after create callback, override in subclasses and don't forget to
+  # call `super()`.
+  # @params [Hash] attrs attributes used on initialization
+  def after_create(attrs)
+    self.class.members.add(id)
+  end
+
+  # Cheap before destroy callback, override in subclasses and don't forget to
+  # call `super()`.
+  def before_destroy
+    lookups # Build lookups
+  end
+
+  # Cheap after destroy callback, override in subclasses and don't forget to
+  # call `super()`. Called within the transaction, be careful not to include
+  # read opertions.
+  def after_destroy
+  end
+
+  protected :store, :store_attribute, :after_create, :before_destroy, :after_destroy
   private   :fetch, :delete, :delete_if, :keep_if, :merge!, :reject!, :select!, :replace
 
 end

data/lib/redpear/store/hash.rb

@@ -108,7 +108,7 @@ class Redpear::Store::Hash < Redpear::Store::Enumerable
     hash = hash.reject do |field, value|
       delete(field) if value.nil?
     end
-    conn.hmset key, *hash.flatten
+    conn.hmset key, *hash.flatten unless hash.empty?
   end
 
   # Comparator

data/lib/redpear/version.rb

@@ -2,7 +2,7 @@ module Redpear
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 7
-    TINY  = 3
+    TINY  = 4
     PRE   = nil
 
     STRING = [MAJOR, MINOR, TINY, PRE].compact.join('.')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redpear
 version: !ruby/object:Gem::Version
-  version: 0.7.3
+  version: 0.7.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-02-28 00:00:00.000000000 Z
+date: 2012-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
-  requirement: &10021500 !ruby/object:Gem::Requirement
+  requirement: &13791300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 2.2.0
   type: :runtime
   prerelease: false
-  version_requirements: *10021500
+  version_requirements: *13791300
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &10018400 !ruby/object:Gem::Requirement
+  requirement: &13790900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10018400
+  version_requirements: *13790900
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &10032440 !ruby/object:Gem::Requirement
+  requirement: &13807720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10032440
+  version_requirements: *13807720
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &10031460 !ruby/object:Gem::Requirement
+  requirement: &13807020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10031460
+  version_requirements: *13807020
 - !ruby/object:Gem::Dependency
   name: shoulda-matchers
-  requirement: &10030280 !ruby/object:Gem::Requirement
+  requirement: &13806580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10030280
+  version_requirements: *13806580
 - !ruby/object:Gem::Dependency
   name: machinist
-  requirement: &10029100 !ruby/object:Gem::Requirement
+  requirement: &13806160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10029100
+  version_requirements: *13806160
 - !ruby/object:Gem::Dependency
   name: factory_girl
-  requirement: &10027760 !ruby/object:Gem::Requirement
+  requirement: &13805500 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10027760
+  version_requirements: *13805500
 - !ruby/object:Gem::Dependency
   name: hiredis
-  requirement: &10026860 !ruby/object:Gem::Requirement
+  requirement: &13804860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *10026860
+  version_requirements: *13804860
 description: Simple, elegant & efficient ORM for Redis
 email: dimitrij@blacksquaremedia.com
 executables: []