codebrulee-attr_remote 0.0.2 → 0.0.3

data/attr_remote.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name     = "attr_remote"
-  s.version  = "0.0.2"
+  s.version  = "0.0.3"
   s.date     = "2008-01-06"
   s.summary  = "Painlessly integrate ActiveResource into ActiveRecord."
   s.email    = "smithk14@gmail.com"

data/lib/attr_remote.rb

@@ -64,6 +64,55 @@ module AttrRemote
         
       # def validate_remote_user_on_create; end  
         def validate_#{remote_instance_meth}_on_create; end
+        
+      # before_update :update_remote_user  
+        before_update :update_#{remote_instance_meth}
+        
+      # def update_remote_user
+      #   if self.remote_user_id
+      #     remote_hash = {}
+      #     self.class.remote_attributes.each do |attr|
+      #       remote_hash[attr.to_sym] = self.send(attr.to_sym)
+      #     end
+      #     remote_user.load(remote_hash)
+      #     remote_user.save
+      #     
+      #     unless @#{remote_instance_meth}.valid?
+      #       remote_user.errors.each do |attr, err|
+      #         errors.add(attr, err)
+      #       end
+      #       return false
+      #     else
+      #       return true
+      #     end
+      #   end
+      # end
+        def update_#{remote_instance_meth}
+          if self.#{remote_instance_id}
+            remote_hash = {}
+            self.class.remote_attributes.each do |attr|
+              remote_hash[attr.to_sym] = self.send(attr.to_sym)
+            end
+            #{remote_instance_meth}.load(remote_hash)
+            #{remote_instance_meth}.save
+            
+            unless @#{remote_instance_meth}.valid?
+              #{remote_instance_meth}.errors.each do |attr, err|
+                errors.add(attr, err)
+              end
+              return false
+            else
+              return true
+            end
+          end
+        end
+        private :update_#{remote_instance_meth}
+        
+      # validate_on_update :validate_remote_user_on_update
+        validate_on_update :validate_#{remote_instance_meth}_on_update
+        
+      # def validate_remote_user_on_update; end  
+        def validate_#{remote_instance_meth}_on_update; end
       remote_access
     
       @remote_attributes = remote_attrs
@@ -79,7 +128,7 @@ module AttrRemote
             elsif self.#{remote_instance_meth}    #   elsif self.remote_user
               @#{attr} = self.                    #     @username = self.
                          #{remote_instance_meth}. #                 remote_user.
-                         #{attr}                  #                 username
+                         #{attr} rescue nil       #                 username rescue nil
             else                                  #   else
               nil                                 #     nil
             end                                   #   end

data/test/helper.rb

@@ -29,6 +29,8 @@ ActiveResource::HttpMock.respond_to do |mock|
     'Location' => 'http://0.0.0.0:3000/users/1'
   }
   
+  mock.put "/users/1.xml", {}, '', 200, {}
+  
   # bob is not so lucky
   mock.post "/bobs.xml", 
             {}, 

data/test/models/user.rb

@@ -1,3 +1,3 @@
 class User < ActiveRecord::Base
-  attr_remote :name
+  attr_remote :name, :not_here
 end

data/test/test_attr_remote.rb

@@ -35,6 +35,12 @@ class TestAttrRemote < Test::Unit::TestCase
       user = Factory.build(:user, :remote_user_id => 666, :name => nil)
       assert_equal '', user.name
     end
+    
+    test "that an attribute that is not readable returns an empty string" do
+      user = Factory(:user)
+      user = User.find(user.id)
+      assert_equal '', user.not_here
+    end
   end
   
   context "creation" do
@@ -66,6 +72,18 @@ class TestAttrRemote < Test::Unit::TestCase
   end
   
   context "updates" do
+    test "that an attribute change is propogated" do
+      user = Factory(:user)
+      user.name = "boo"
+      assert user.save
+      ActiveResource::HttpMock.respond_to do |mock|
+        mock.get "/users/1.xml", {}, {
+          :name => "boo"
+        }.to_xml(:root => "user")
+      end
+      user = User.find(user.id)
+      assert_equal "boo", user.name
+    end
   end
   
   context "deletes" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: codebrulee-attr_remote
 version: !ruby/object:Gem::Version 
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors: 
 - Kevin Smith