reactive_resource 0.6.0 → 0.6.1

data/lib/reactive_resource/association/belongs_to_association.rb

@@ -55,12 +55,13 @@ module ReactiveResource
           # address.lawyer_id
           define_method("#{attribute}_id") do
             prefix_options["#{attribute}_id".intern]
+            super()
           end
           
           # address.lawyer_id = 3
           define_method("#{attribute}_id=") do |value|
             prefix_options["#{attribute}_id".intern] = value
-            attributes["#{attribute}_id".intern] = value
+            super(value)
           end
 
           # address.lawyer

data/lib/reactive_resource/base.rb

@@ -89,13 +89,14 @@ module ReactiveResource
     # be in the URL anyway), so we'll try to inject them based on the
     # attributes of the object we just used.
     def load(attributes)
+      attributes = attributes.stringify_keys
       self.class.belongs_to_with_parents.each do |belongs_to_param|
-        attributes["#{belongs_to_param}_id".intern] ||= prefix_options["#{belongs_to_param}_id".intern]
+        attributes["#{belongs_to_param}_id"] ||= prefix_options["#{belongs_to_param}_id".intern]
 
         # also set prefix attributes as real attributes. Otherwise,
         # belongs_to attributes will be stripped out of the response
         # even if we aren't actually using the association.
-        @attributes["#{belongs_to_param}_id".intern] = attributes["#{belongs_to_param}_id".intern]
+        @attributes["#{belongs_to_param}_id"] = attributes["#{belongs_to_param}_id"]
       end
       super(attributes)
     end

data/lib/reactive_resource/version.rb

@@ -1,4 +1,4 @@
 module ReactiveResource
   # The current version of ReactiveResource
-  VERSION = "0.6.0"
+  VERSION = "0.6.1"
 end

data/test/unit/base_test.rb

@@ -175,11 +175,33 @@ class ReactiveResource::BaseTest < Test::Unit::TestCase
       assert_requested(:post, "https://api.avvo.com/api/1/lawyers/1/lawyer_posts.json")
     end
 
+    should "treat returned prefix parameters as regular parameters" do
+      stub_request(:get, "https://api.avvo.com/api/1/lawyers/1/lawyer_posts.json").
+        to_return(:body =>[{"id" => 1, "post_id" => 2}].to_json)
+      stub_request(:put, "https://api.avvo.com/api/1/lawyers/1/lawyer_posts/1.json").
+        with(:body => {"lawyer_post" => {"id" => 1, "post_id" => 3}})
+      
+      @object = ReactiveResource::LawyerPost.find(:all, :params => {:lawyer_id => 1}).first
+      assert_not_nil @object.post_id
+
+      @object.post_id = 3
+      @object.save
+
+      assert_requested(:put, "https://api.avvo.com/api/1/lawyers/1/lawyer_posts/1.json")
+    end
+
     should "set the attributes correctly when it's created" do
       @object = ReactiveResource::LawyerPost.new(:lawyer_id => 1, :post_id => 2)
       assert_not_nil @object.attributes[:post_id]
       assert_not_nil @object.attributes[:lawyer_id]
     end
+
+    should "set the attributes correctly when it's created with strings" do
+      @object = ReactiveResource::LawyerPost.new("lawyer_id" => 1, "post_id" => 2)
+      assert_not_nil @object.post_id
+      assert_not_nil @object.attributes["post_id"]
+      assert_not_nil @object.attributes[:lawyer_id]
+    end
   end
   
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 6
-  - 0
-  version: 0.6.0
+  - 1
+  version: 0.6.1
 platform: ruby
 authors: 
 - Justin Weiss
@@ -14,11 +14,12 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-13 00:00:00 -07:00
+date: 2011-04-14 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: activeresource
+  prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -30,10 +31,10 @@ dependencies:
         - 10
         version: 2.3.10
   type: :runtime
-  prerelease: false
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
   name: shoulda
+  prerelease: false
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -45,10 +46,10 @@ dependencies:
         - 3
         version: 2.11.3
   type: :development
-  prerelease: false
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
   name: webmock
+  prerelease: false
   requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
@@ -60,7 +61,6 @@ dependencies:
         - 1
         version: 1.6.1
   type: :development
-  prerelease: false
   version_requirements: *id003
 description: ActiveRecord-like associations for ActiveResource
 email: 
@@ -105,7 +105,6 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -2570327779021755250
       segments: 
       - 0
       version: "0"
@@ -114,7 +113,6 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -2570327779021755250
       segments: 
       - 0
       version: "0"