activeresource 5.0.0 → 5.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: f09e2bfd3d8332924184d674556d18d2d0533dcf
-  data.tar.gz: 5dbc9edf5003415fd3565692dffa8a6342d0a1b2
+SHA256:
+  metadata.gz: 5ba8858054bdc522433ffd05850eb3246ef21e3aee7ffab4aec2590ebe8eedfd
+  data.tar.gz: e52b3454a54fb9c6a97176da0464556ae645a3f7e0a8370fb0342dc3e9795eb1
 SHA512:
-  metadata.gz: 27ba6aabd0eea02c3f2adb1d6d92311c94415d9ac32589f24abc517f1384a0ab6471717ab50e017a4993b069452f85e636053e3e49ffeb37c60873fcc4f9f671
-  data.tar.gz: 3c8c8780b50ada3328e3df5b0d544e19cc3977dd4199c750991f403e00cfaa9c45213328cfdc717ebb0d86bbd931bc8b48724f17d7c56d7f996c6ace763e0ad6
+  metadata.gz: f716f552fb861f8aadc6ddcaf5265076d905d0428b3fdb368b52a21030b1ba33acd148c8dd548801f5f7fe2cf13a3c94055079ae22a378d9ad26b56cb244df3a
+  data.tar.gz: d7103ae2decdd750363b186ef5895eb18a04f4450b9815f98b74d0391445acaa3eae1084a53f0bff1774be7dca88cadd19bde4849f864a093bb6213d06879da9

data/MIT-LICENSE

@@ -0,0 +1,20 @@
+Copyright (c) 2006-2016 David Heinemeier Hansson
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.rdoc

@@ -30,10 +30,6 @@ Or added to a Gemfile:
 
   gem 'activeresource'
 
-For compatibility with Rails 5, use:
-
-  gem 'activeresource', github: 'rails/activeresource', branch: 'master'
-
 Source code can be downloaded on GitHub
 
 * https://github.com/rails/activeresource/tree/master/activeresource
@@ -57,6 +53,11 @@ life cycle methods that operate against a persistent store.
 As you can see, the methods are quite similar to Active Record's methods for dealing with database
 records.  But rather than dealing directly with a database record, you're dealing with HTTP resources (which may or may not be database records).
 
+Connection settings (`site`, `headers`, `user`, `password`, `proxy`) and the connections themselves are store in
+thread-local variables to make them thread-safe, so you can also set these dynamically, even in a multi-threaded environment, for instance:
+
+    ActiveResource::Base.site = api_site_for(request)
+
 ==== Authentication
 
 Active Resource supports the token based authentication provided by Rails through the <tt>ActionController::HttpAuthentication::Token</tt> class using custom headers.
@@ -64,8 +65,32 @@ Active Resource supports the token based authentication provided by Rails throug
    class Person < ActiveResource::Base
      self.headers['Authorization'] = 'Token token="abcd"'
    end
-   
-You can also set any specific HTTP header using the same way.
+
+You can also set any specific HTTP header using the same way.  As mentioned above, headers are thread-safe, so you can set headers dynamically, even in a multi-threaded environment:
+
+    ActiveResource::Base.headers['Authorization'] = current_session_api_token
+    
+Global Authentication to be used across all subclasses of ActiveResource::Base should be handled using the ActiveResource::Connection class.
+
+    ActiveResource::Base.connection.auth_type = :bearer
+    ActiveResource::Base.connection.bearer_token = @bearer_token
+    
+    class Person < ActiveResource::Base
+      self.connection.auth_type = :bearer
+      self.connection.bearer_token = @bearer_token
+    end
+
+ActiveResource supports 2 options for HTTP authentication today.
+
+1. Basic
+
+    ActiveResource::Connection.new("http://my%40email.com:%31%32%33@localhost")
+    # username: my@email.com password: 123
+
+2. Bearer Token
+
+    ActiveResource::Base.connection.auth_type = :bearer
+    ActiveResource::Base.connection.bearer_token = @bearer_token
 
 ==== Protocol
 
@@ -115,7 +140,7 @@ Collections can also be requested in a similar fashion
 
    # Expects a response of
    #
-   # [ 
+   # [
    #   {"id":1,"first":"Tyler","last":"Durden"},
    #   {"id":2,"first":"Tony","last":"Stark",}
    # ]
@@ -205,7 +230,7 @@ class definition below:
   end
 
   post = Post.find(1)      # issues GET http://blog.io/posts/1.json
-  comments = post.comments # issues GET http://blog.io/posts/1/comments.json
+  comments = post.comments # issues GET http://blog.io/comments.json?post_id=1
 
 
 If you control the server, you may wish to include nested resources thus avoiding a
@@ -221,6 +246,19 @@ The server-side model can be adjusted as follows to include comments in the resp
     end
   end
 
+==== Logging
+
+Active Resource instruments the event `request.active_resource` when doing a request
+to the remote service. You can subscribe to it by doing:
+
+   ActiveSupport::Notifications.subscribe('request.active_resource')  do |name, start, finish, id, payload|
+
+The `payload` is a `Hash` with the following keys:
+
+* `method` as a `Symbol`
+* `request_uri` as a `String`
+* `result` as an `Net::HTTPResponse`
+
 == License
 
 Active Resource is released under the MIT license:
@@ -236,9 +274,9 @@ See {CONTRIBUTING}[https://github.com/rails/activeresource/blob/master/CONTRIBUT
 
 == Support
 
-API documentation is at
+Full API documentation is available at
 
-* http://rubydoc.info/gems/activeresource/4.0.0/frames
+* http://rubydoc.info/gems/activeresource
 
 Bug reports and feature requests can be filed with the rest for the Ruby on Rails project here:
 

data/lib/active_resource.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 #--
 # Copyright (c) 2006-2012 David Heinemeier Hansson
 #
@@ -21,10 +23,10 @@
 # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #++
 
-require 'active_support'
-require 'active_model'
-require 'active_resource/exceptions'
-require 'active_resource/version'
+require "active_support"
+require "active_model"
+require "active_resource/exceptions"
+require "active_resource/version"
 
 module ActiveResource
   extend ActiveSupport::Autoload
@@ -41,4 +43,4 @@ module ActiveResource
   autoload :Collection
 end
 
-require 'active_resource/railtie' if defined?(Rails.application)
+require "active_resource/railtie" if defined?(Rails.application)

data/lib/active_resource/active_job_serializer.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module ActiveResource
+  class ActiveJobSerializer < ActiveJob::Serializers::ObjectSerializer
+    def serialize(resource)
+      super(
+        "class" => resource.class.name,
+        "persisted" => resource.persisted?,
+        "prefix_options" => resource.prefix_options.as_json,
+        "attributes" => resource.attributes.as_json
+      )
+    end
+
+    def deserialize(hash)
+      hash["class"].constantize.new(hash["attributes"]).tap do |resource|
+        resource.persisted      = hash["persisted"]
+        resource.prefix_options = hash["prefix_options"]
+      end
+    end
+
+    private
+      def klass
+        ActiveResource::Base
+      end
+  end
+end

data/lib/active_resource/associations.rb

@@ -1,10 +1,11 @@
-module ActiveResource::Associations
+# frozen_string_literal: true
 
+module ActiveResource::Associations
   module Builder
-    autoload :Association, 'active_resource/associations/builder/association'
-    autoload :HasMany,     'active_resource/associations/builder/has_many'
-    autoload :HasOne,      'active_resource/associations/builder/has_one'
-    autoload :BelongsTo,   'active_resource/associations/builder/belongs_to'
+    autoload :Association, "active_resource/associations/builder/association"
+    autoload :HasMany,     "active_resource/associations/builder/has_many"
+    autoload :HasOne,      "active_resource/associations/builder/has_one"
+    autoload :BelongsTo,   "active_resource/associations/builder/belongs_to"
   end
 
 
@@ -112,7 +113,7 @@ module ActiveResource::Associations
   # <tt>belongs_to :customer, :foreign_key => 'user_id'</tt>
   # Creates a belongs_to association called customer which would be resolved by the foreign_key <tt>user_id</tt> instead of <tt>customer_id</tt>
   #
-  def belongs_to(name, options={})
+  def belongs_to(name, options = {})
     Builder::BelongsTo.build(self, name, options)
   end
 
@@ -147,7 +148,7 @@ module ActiveResource::Associations
       elsif attributes.include?(method_name)
         attributes[method_name]
       elsif !new_record?
-        instance_variable_set(ivar_name, reflection.klass.find(:all, :params => {:"#{self.class.element_name}_id" => self.id}))
+        instance_variable_set(ivar_name, reflection.klass.find(:all, params: { :"#{self.class.element_name}_id" => self.id }))
       else
         instance_variable_set(ivar_name, self.class.collection_parser.new)
       end
@@ -165,11 +166,10 @@ module ActiveResource::Associations
       elsif attributes.include?(method_name)
         attributes[method_name]
       elsif reflection.klass.respond_to?(:singleton_name)
-        instance_variable_set(ivar_name, reflection.klass.find(:params => {:"#{self.class.element_name}_id" => self.id}))
+        instance_variable_set(ivar_name, reflection.klass.find(params: { :"#{self.class.element_name}_id" => self.id }))
       else
-        instance_variable_set(ivar_name, reflection.klass.find(:one, :from => "/#{self.class.collection_name}/#{self.id}/#{method_name}#{self.class.format_extension}"))
+        instance_variable_set(ivar_name, reflection.klass.find(:one, from: "/#{self.class.collection_name}/#{self.id}/#{method_name}#{self.class.format_extension}"))
       end
     end
   end
-
 end

data/lib/active_resource/associations/builder/association.rb

@@ -1,6 +1,7 @@
+# frozen_string_literal: true
+
 module ActiveResource::Associations::Builder
   class Association #:nodoc:
-
     # providing a Class-Variable, which will have a different store of subclasses
     class_attribute :valid_options
     self.valid_options = [:class_name]
@@ -25,8 +26,8 @@ module ActiveResource::Associations::Builder
 
     private
 
-    def validate_options
-      options.assert_valid_keys(self.class.valid_options)
-    end
+      def validate_options
+        options.assert_valid_keys(self.class.valid_options)
+      end
   end
 end

data/lib/active_resource/associations/builder/belongs_to.rb

@@ -1,4 +1,6 @@
-module ActiveResource::Associations::Builder 
+# frozen_string_literal: true
+
+module ActiveResource::Associations::Builder
   class BelongsTo < Association
     self.valid_options += [:foreign_key]
 
@@ -8,7 +10,7 @@ module ActiveResource::Associations::Builder
       validate_options
       reflection = model.create_reflection(self.class.macro, name, options)
       model.defines_belongs_to_finder_method(reflection)
-      return reflection
+      reflection
     end
   end
 end

data/lib/active_resource/associations/builder/has_many.rb

@@ -1,4 +1,6 @@
-module ActiveResource::Associations::Builder 
+# frozen_string_literal: true
+
+module ActiveResource::Associations::Builder
   class HasMany < Association
     self.macro = :has_many
 

data/lib/active_resource/associations/builder/has_one.rb

@@ -1,7 +1,9 @@
-module ActiveResource::Associations::Builder 
+# frozen_string_literal: true
+
+module ActiveResource::Associations::Builder
   class HasOne < Association
     self.macro = :has_one
-    
+
     def build
       validate_options
       model.create_reflection(self.class.macro, name, options).tap do |reflection|

data/lib/active_resource/base.rb

@@ -1,26 +1,28 @@
-require 'active_support'
-require 'active_support/core_ext/class/attribute_accessors'
-require 'active_support/core_ext/class/attribute'
-require 'active_support/core_ext/hash/indifferent_access'
-require 'active_support/core_ext/kernel/reporting'
-require 'active_support/core_ext/module/delegation'
-require 'active_support/core_ext/module/aliasing'
-require 'active_support/core_ext/object/blank'
-require 'active_support/core_ext/object/to_query'
-require 'active_support/core_ext/object/duplicable'
-require 'set'
-require 'uri'
-
-require 'active_support/core_ext/uri'
-require 'active_resource/connection'
-require 'active_resource/formats'
-require 'active_resource/schema'
-require 'active_resource/log_subscriber'
-require 'active_resource/associations'
-require 'active_resource/reflection'
-require 'active_resource/threadsafe_attributes'
-
-require 'active_model/serializers/xml'
+# frozen_string_literal: true
+
+require "active_support"
+require "active_support/core_ext/class/attribute_accessors"
+require "active_support/core_ext/class/attribute"
+require "active_support/core_ext/hash/indifferent_access"
+require "active_support/core_ext/kernel/reporting"
+require "active_support/core_ext/module/delegation"
+require "active_support/core_ext/module/aliasing"
+require "active_support/core_ext/object/blank"
+require "active_support/core_ext/object/to_query"
+require "active_support/core_ext/object/duplicable"
+require "set"
+require "uri"
+
+require "active_support/core_ext/uri"
+require "active_resource/connection"
+require "active_resource/formats"
+require "active_resource/schema"
+require "active_resource/log_subscriber"
+require "active_resource/associations"
+require "active_resource/reflection"
+require "active_resource/threadsafe_attributes"
+
+require "active_model/serializers/xml"
 
 module ActiveResource
   # ActiveResource::Base is the main class for mapping RESTful resources as models in a Rails application.
@@ -300,7 +302,12 @@ module ActiveResource
     ##
     # :singleton-method:
     # The logger for diagnosing and tracing Active Resource calls.
-    cattr_accessor :logger
+    cattr_reader :logger
+
+    def self.logger=(logger)
+      self._connection = nil
+      @@logger = logger
+    end
 
     class_attribute :_format
     class_attribute :_collection_parser
@@ -378,7 +385,7 @@ module ActiveResource
           @schema ||= {}.with_indifferent_access
           @known_attributes ||= []
 
-          schema_definition.attrs.each do |k,v|
+          schema_definition.attrs.each do |k, v|
             @schema[k] = v
             @known_attributes << k
           end
@@ -417,7 +424,7 @@ module ActiveResource
         raise ArgumentError, "Expected a hash" unless the_schema.kind_of? Hash
 
         schema do
-          the_schema.each {|k,v| attribute(k,v) }
+          the_schema.each { |k, v| attribute(k, v) }
         end
       end
 
@@ -619,7 +626,7 @@ module ActiveResource
       # * <tt>:ssl_timeout</tt> -The SSL timeout in seconds.
       def ssl_options=(options)
         self._connection = nil
-        @ssl_options  = options
+        @ssl_options = options
       end
 
       # Returns the SSL options hash.
@@ -636,7 +643,9 @@ module ActiveResource
       # or not (defaults to <tt>false</tt>).
       def connection(refresh = false)
         if _connection_defined? || superclass == Object
-          self._connection = connection_class.new(site, format) if refresh || _connection.nil?
+          self._connection = connection_class.new(
+            site, format, logger: logger
+          ) if refresh || _connection.nil?
           _connection.proxy = proxy if proxy
           _connection.user = user if user
           _connection.password = password if password
@@ -682,15 +691,15 @@ module ActiveResource
           return primary_key if primary_key.is_a?(Symbol)
           primary_key.dup.freeze
         else
-          'id'
+          "id"
         end
       end
 
       # Gets the \prefix for a resource's nested URL (e.g., <tt>prefix/collectionname/1.json</tt>)
       # This method is regenerated at runtime based on what the \prefix is set to.
-      def prefix(options={})
+      def prefix(options = {})
         default = site.path
-        default << '/' unless default[-1..-1] == '/'
+        default << "/" unless default[-1..-1] == "/"
         # generate the actual method based on the current site path
         self.prefix = default
         prefix(options)
@@ -705,7 +714,7 @@ module ActiveResource
 
       # Sets the \prefix for a resource's nested URL (e.g., <tt>prefix/collectionname/1.json</tt>).
       # Default value is <tt>site.path</tt>.
-      def prefix=(value = '/')
+      def prefix=(value = "/")
         # Replace :placeholders with '#{embedded options[:lookups]}'
         prefix_call = value.gsub(/:\w+/) { |key| "\#{URI.parser.escape options[#{key}].to_s}" }
 
@@ -766,6 +775,36 @@ module ActiveResource
         "#{prefix(prefix_options)}#{collection_name}/#{URI.parser.escape id.to_s}#{format_extension}#{query_string(query_options)}"
       end
 
+      # Gets the element url for the given ID in +id+. If the +query_options+ parameter is omitted, Rails
+      # will split from the \prefix options.
+      #
+      # ==== Options
+      # +prefix_options+ - A \hash to add a \prefix to the request for nested URLs (e.g., <tt>:account_id => 19</tt>
+      # would yield a URL like <tt>https://37s.sunrise.com/accounts/19/purchases.json</tt>).
+      #
+      # +query_options+ - A \hash to add items to the query string for the request.
+      #
+      # ==== Examples
+      #   Post.element_url(1)
+      #   # => https://37s.sunrise.com/posts/1.json
+      #
+      #   class Comment < ActiveResource::Base
+      #     self.site = "https://37s.sunrise.com/posts/:post_id"
+      #   end
+      #
+      #   Comment.element_url(1, :post_id => 5)
+      #   # => https://37s.sunrise.com/posts/5/comments/1.json
+      #
+      #   Comment.element_url(1, :post_id => 5, :active => 1)
+      #   # => https://37s.sunrise.com/posts/5/comments/1.json?active=1
+      #
+      #   Comment.element_url(1, {:post_id => 5}, {:active => 1})
+      #   # => https://37s.sunrise.com/posts/5/comments/1.json?active=1
+      #
+      def element_url(id, prefix_options = {}, query_options = nil)
+        URI.join(site, element_path(id, prefix_options, query_options)).to_s
+      end
+
       # Gets the new element path for REST resources.
       #
       # ==== Options
@@ -931,11 +970,18 @@ module ActiveResource
         options = arguments.slice!(0) || {}
 
         case scope
-          when :all   then find_every(options)
-          when :first then find_every(options).to_a.first
-          when :last  then find_every(options).to_a.last
-          when :one   then find_one(options)
-          else             find_single(scope, options)
+        when :all
+          find_every(options)
+        when :first
+          collection = find_every(options)
+          collection && collection.first
+        when :last
+          collection = find_every(options)
+          collection && collection.last
+        when :one
+          find_one(options)
+        else
+          find_single(scope, options)
         end
       end
 
@@ -962,7 +1008,7 @@ module ActiveResource
 
       def where(clauses = {})
         raise ArgumentError, "expected a clauses Hash, got #{clauses.inspect}" unless clauses.is_a? Hash
-        find(:all, :params => clauses)
+        find(:all, params: clauses)
       end
 
 
@@ -996,7 +1042,7 @@ module ActiveResource
           prefix_options, query_options = split_options(options[:params])
           path = element_path(id, prefix_options, query_options)
           response = connection.head(path, headers)
-          (200..206).include? response.code
+          (200..206).include? response.code.to_i
         end
         # id && !find_single(id, options).nil?
       rescue ActiveResource::ResourceNotFound, ActiveResource::ResourceGone
@@ -1024,7 +1070,7 @@ module ActiveResource
             else
               prefix_options, query_options = split_options(options[:params])
               path = collection_path(prefix_options, query_options)
-              instantiate_collection( (format.decode(connection.get(path, headers).body) || []), query_options, prefix_options )
+              instantiate_collection((format.decode(connection.get(path, headers).body) || []), query_options, prefix_options)
             end
           rescue ActiveResource::ResourceNotFound
             # Swallowing ResourceNotFound exceptions and return nil - as per
@@ -1091,8 +1137,8 @@ module ActiveResource
           prefix_options, query_options = {}, {}
 
           (options || {}).each do |key, value|
-            next if key.blank? || !key.respond_to?(:to_sym)
-            (prefix_parameters.include?(key.to_sym) ? prefix_options : query_options)[key.to_sym] = value
+            next if key.blank?
+            (prefix_parameters.include?(key.to_s.to_sym) ? prefix_options : query_options)[key.to_s.to_sym] = value
           end
 
           [ prefix_options, query_options ]
@@ -1157,13 +1203,13 @@ module ActiveResource
     #   not_ryan.hash            # => {:not => "an ARes instance"}
     def clone
       # Clone all attributes except the pk and any nested ARes
-      cloned = Hash[attributes.reject {|k,v| k == self.class.primary_key || v.is_a?(ActiveResource::Base)}.map { |k, v| [k, v.clone] }]
+      cloned = Hash[attributes.reject { |k, v| k == self.class.primary_key || v.is_a?(ActiveResource::Base) }.map { |k, v| [k, v.clone] }]
       # Form the new resource - bypass initialize of resource with 'new' as that will call 'load' which
       # attempts to convert hashes into member objects and arrays into collections of objects. We want
       # the raw objects to be cloned so we bypass load by directly setting the attributes hash.
       resource = self.class.new({})
       resource.prefix_options = self.prefix_options
-      resource.send :instance_variable_set, '@attributes', cloned
+      resource.send :instance_variable_set, "@attributes", cloned
       resource
     end
 
@@ -1338,13 +1384,13 @@ module ActiveResource
     #   Person.delete(guys_id)
     #   that_guy.exists? # => false
     def exists?
-      !new? && self.class.exists?(to_param, :params => prefix_options)
+      !new? && self.class.exists?(to_param, params: prefix_options)
     end
 
     # Returns the serialized string representation of the resource in the configured
     # serialization format specified in ActiveResource::Base.format. The options
     # applicable depend on the configured encoding format.
-    def encode(options={})
+    def encode(options = {})
       send("to_#{self.class.format.extension}", options)
     end
 
@@ -1360,7 +1406,7 @@ module ActiveResource
     #   my_branch.reload
     #   my_branch.name # => "Wilson Road"
     def reload
-      self.load(self.class.find(to_param, :params => @prefix_options).attributes, false, true)
+      self.load(self.class.find(to_param, params: @prefix_options).attributes, false, true)
     end
 
     # A method to manually load attributes from a \hash. Recursively loads collections of
@@ -1401,21 +1447,21 @@ module ActiveResource
       attributes.each do |key, value|
         @attributes[key.to_s] =
           case value
-            when Array
-              resource = nil
-              value.map do |attrs|
-                if attrs.is_a?(Hash)
-                  resource ||= find_or_create_resource_for_collection(key)
-                  resource.new(attrs, persisted)
-                else
-                  attrs.duplicable? ? attrs.dup : attrs
-                end
+          when Array
+            resource = nil
+            value.map do |attrs|
+              if attrs.is_a?(Hash)
+                resource ||= find_or_create_resource_for_collection(key)
+                resource.new(attrs, persisted)
+              else
+                attrs.duplicable? ? attrs.dup : attrs
               end
-            when Hash
-              resource = find_or_create_resource_for(key)
-              resource.new(value, persisted)
-            else
-              value.duplicable? ? value.dup : value
+            end
+          when Hash
+            resource = find_or_create_resource_for(key)
+            resource.new(value, persisted)
+          else
+            value.duplicable? ? value.dup : value
           end
       end
       self
@@ -1472,12 +1518,20 @@ module ActiveResource
       end
     end
 
-    def to_json(options={})
-      super(include_root_in_json ? { :root => self.class.element_name }.merge(options) : options)
+    def to_json(options = {})
+      super(include_root_in_json ? { root: self.class.element_name }.merge(options) : options)
     end
 
-    def to_xml(options={})
-      super({ :root => self.class.element_name }.merge(options))
+    def to_xml(options = {})
+      super({ root: self.class.element_name }.merge(options))
+    end
+
+    def read_attribute_for_serialization(n)
+      if !attributes[n].nil?
+        attributes[n]
+      elsif respond_to?(n)
+        send(n)
+      end
     end
 
     protected
@@ -1505,9 +1559,9 @@ module ActiveResource
       end
 
       def load_attributes_from_response(response)
-        if (response_code_allows_body?(response.code) &&
-            (response['Content-Length'].nil? || response['Content-Length'] != "0") &&
-            !response.body.nil? && response.body.strip.size > 0)
+        if response_code_allows_body?(response.code.to_i) &&
+            (response["Content-Length"].nil? || response["Content-Length"] != "0") &&
+            !response.body.nil? && response.body.strip.size > 0
           load(self.class.format.decode(response.body), true, true)
           @persisted = true
         end
@@ -1515,13 +1569,17 @@ module ActiveResource
 
       # Takes a response from a typical create post and pulls the ID out
       def id_from_response(response)
-        response['Location'][/\/([^\/]*?)(\.\w+)?$/, 1] if response['Location']
+        response["Location"][/\/([^\/]*?)(\.\w+)?$/, 1] if response["Location"]
       end
 
       def element_path(options = nil)
         self.class.element_path(to_param, options || prefix_options)
       end
 
+      def element_url(options = nil)
+        self.class.element_url(to_param, options || prefix_options)
+      end
+
       def new_element_path
         self.class.new_element_path(prefix_options)
       end
@@ -1532,13 +1590,9 @@ module ActiveResource
 
     private
 
-      def read_attribute_for_serialization(n)
-        attributes[n]
-      end
-
       # Determine whether the response is allowed to have a body per HTTP 1.1 spec section 4.4.1
       def response_code_allows_body?(c)
-        !((100..199).include?(c) || [204,304].include?(c))
+        !((100..199).include?(c) || [204, 304].include?(c))
       end
 
       # Tries to find a resource for a given collection name; if it fails, then the resource is created
@@ -1551,7 +1605,7 @@ module ActiveResource
       # if it fails, then the resource is created
       def find_or_create_resource_in_modules(resource_name, module_names)
         receiver = Object
-        namespaces = module_names[0, module_names.size-1].map do |module_name|
+        namespaces = module_names[0, module_names.size - 1].map do |module_name|
           receiver = receiver.const_get(module_name)
         end
         const_args = [resource_name, false]
@@ -1568,7 +1622,11 @@ module ActiveResource
         resource_name = name.to_s.camelize
 
         const_args = [resource_name, false]
-        if self.class.const_defined?(*const_args)
+
+        if !const_valid?(*const_args)
+          # resource_name is not a valid ruby module name and cannot be created normally
+          find_or_create_resource_for(:UnnamedResource)
+        elsif self.class.const_defined?(*const_args)
           self.class.const_get(*const_args)
         else
           ancestors = self.class.name.to_s.split("::")
@@ -1584,6 +1642,13 @@ module ActiveResource
         end
       end
 
+      def const_valid?(*const_args)
+        self.class.const_defined?(*const_args)
+        true
+      rescue NameError
+        false
+      end
+
       # Create and return a class definition for a resource inside the current resource
       def create_resource_for(resource_name)
         resource = self.class.const_set(resource_name, Class.new(ActiveResource::Base))