passive_record 0.2.2 → 0.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cef72fdda85e2a5d1dabfd009842c092780ce77d
-  data.tar.gz: a6f81fbd8c7a0b88d318e46a8aa232ae69b0a549
+  metadata.gz: 44c674c3b4dc0121e16a6368fce9d4109ccf6b51
+  data.tar.gz: 7621a7ffbba7ee1cf08cd36bcd0cb053d36da13f
 SHA512:
-  metadata.gz: bb22e938cd0081234dd6f02b02ae060d3fc37e794b78b479a313e6793ab0fe486f367dfa38c91abb98b1f2594297a33ae4b529968d8b3906c1addcaacea6c8a1
-  data.tar.gz: d0e3ee535809bb7817be14dece26cb727bb0db7621fb1a090f9baa2ad69b9f54366c29a73f4651e15f938e2edf802e424390398212510aa6cd5fcc7d50af4cf1
+  metadata.gz: ea2328b524559ed872dbf095380ecd2d3f4b5fba455c798c600ba35e540965c42929e07c98d8c49e1ce19b456690871cb69bf0e4e3e975f72e43c0cd528af3c3
+  data.tar.gz: 5c12ba3c4eb166561a9c93703dde48e79e0cea00712c3eab9d919591073b16615e8c0f6e905132d56c525e81a46fd6df022e5ee1aa06e1fefa2f8834c714102c

data/bin/passive_record

@@ -8,7 +8,7 @@ if File.directory?(File.join(root,'.git'))
     rescue LoadError => e
       warn e.message
       warn "Run `gem install bundler` to install Bundler"
-      exit -1
+      exit(-1)
     end
   end
 end

data/lib/passive_record/hooks.rb

@@ -31,5 +31,17 @@ module PassiveRecord
       @hooks += [ hook ]
       self
     end
+
+    def after_update_hooks
+      @hooks ||= []
+      @hooks.select { |hook| hook.kind == :after_update }
+    end
+
+    def after_update(*meth_syms, &blk)
+      hook = Hook.new(:after_update,*meth_syms,&blk)
+      @hooks ||= []
+      @hooks += [ hook ]
+      self
+    end
   end
 end

data/lib/passive_record/instance_methods.rb

@@ -2,16 +2,14 @@ module PassiveRecord
   module InstanceMethods
     include PrettyPrinting
 
-    def attribute_names
-      attr_names = instance_variables
-      attr_names += self.class.associations_id_syms
-      attr_names += members rescue []
-      attr_names.reject! { |name| name.to_s.start_with?("@_") }
-      attr_names - blacklisted_attribute_names
-    end
+    def update(attrs={})
+      attrs.each do |k,v|
+        send("#{k}=", v)
+      end
 
-    def blacklisted_attribute_names
-      []
+      self.class.after_update_hooks.each do |hook|
+        hook.run(self)
+      end
     end
 
     # from http://stackoverflow.com/a/8417341/90042
@@ -25,8 +23,6 @@ module PassiveRecord
       ]
     end
 
-    ###
-
     def respond_to?(meth,*args,&blk)
       if find_relation_by_target_name_symbol(meth)
         true
@@ -44,42 +40,26 @@ module PassiveRecord
     end
 
     protected
+    def attribute_names
+      attr_names = instance_variables
+      attr_names += self.class.associations_id_syms
+      attr_names += members rescue []
+      attr_names.reject! { |name| name.to_s.start_with?("@_") }
+      attr_names - blacklisted_attribute_names
+    end
 
-    def send_relation(matching_relation, meth, *args)
-      target_name = matching_relation.association.target_name_symbol.to_s
-
-      case meth.to_s
-      when target_name
-        matching_relation.lookup
-      when "#{target_name}="
-        if args.first.is_a?(Array)
-          # need to loop through each arg and set id
-          args.first.each do |child|
-            child.send(matching_relation.parent_model_id_field + "=", id)
-          end
-        else
-          # assume simple assignment
-          matching_relation.parent_model_id = args.first.id
-        end
-      when "create_#{target_name}", "create_#{target_name.singularize}"
-        matching_relation.create(*args)
-      when "#{target_name}_id"
-        matching_relation.parent_model_id
-      when "#{target_name}_id="
-        matching_relation.parent_model_id = args.first
-      when "#{target_name}_ids", "#{target_name.singularize}_ids"
-        matching_relation.parent_model.send(target_name).map(&:id)
-      end
+    def blacklisted_attribute_names
+      []
     end
 
+    private
+
     def relata
       @_relata ||= self.class.associations&.map do |assn|
         assn.to_relation(self)
       end || []
     end
 
-    private
-
     def find_relation_by_target_name_symbol(meth)
       relata.detect do |relation|  # matching relation...
         possible_target_names(relation).include?(meth.to_s)
@@ -99,5 +79,32 @@ module PassiveRecord
         "create_#{target_name.singularize}"
       ]
     end
+
+    def send_relation(matching_relation, meth, *args)
+      target_name = matching_relation.association.target_name_symbol.to_s
+
+      case meth.to_s
+      when target_name
+        matching_relation.lookup
+      when "#{target_name}="
+        if args.first.is_a?(Array)
+          # need to loop through each arg and set id
+          args.first.each do |child|
+            child.send(matching_relation.parent_model_id_field + "=", id)
+          end
+        else
+          # assume simple assignment
+          matching_relation.parent_model_id = args.first.id
+        end
+      when "create_#{target_name}", "create_#{target_name.singularize}"
+        matching_relation.create(*args)
+      when "#{target_name}_id"
+        matching_relation.parent_model_id
+      when "#{target_name}_id="
+        matching_relation.parent_model_id = args.first
+      when "#{target_name}_ids", "#{target_name.singularize}_ids"
+        matching_relation.parent_model.send(target_name).map(&:id)
+      end
+    end
   end
 end

data/lib/passive_record/version.rb

@@ -1,4 +1,4 @@
 module PassiveRecord
   # passive_record version
-  VERSION = "0.2.2"
+  VERSION = "0.2.3"
 end

data/spec/passive_record_spec.rb

@@ -22,6 +22,17 @@ describe "passive record models" do
     let(:value) { 'foo_value' }
 
     describe "instance methods" do
+      describe "#update" do
+        it 'should update attrs' do
+          expect {model.update(foo: '123')}.to change {model.foo}.from(value).to('123')
+        end
+
+        it 'should invoke callbacks' do
+          model.update(foo: 'barbazquux')
+          expect(model.updated_at).to be_a(Time)
+        end
+      end
+
       describe "#inspect" do
         it 'should report attribute details' do
           expect(model.inspect).to eq("SimpleModel (id: #{model.id.inspect}, foo: \"foo_value\")")

data/spec/spec_helper.rb

@@ -13,6 +13,8 @@ end
 
 class SimpleModel < Struct.new(:foo)
   include PassiveRecord
+  attr_reader :updated_at
+  after_update { @updated_at = Time.now }
 end
 
 module Family

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: passive_record
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
 platform: ruby
 authors:
 - Joseph Weissman