activeresource 4.1.0 → 6.0.0

data/lib/active_resource/base.rb

@@ -1,24 +1,27 @@
-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'
+# 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_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.
@@ -123,18 +126,25 @@ module ActiveResource
   # requests. These sensitive credentials are sent unencrypted, visible to
   # any onlooker, so this scheme should only be used with SSL.
   #
-  # Digest authentication sends a crytographic hash of the username, password,
+  # Digest authentication sends a cryptographic hash of the username, password,
   # HTTP method, URI, and a single-use secret key provided by the server.
   # Sensitive credentials aren't visible to onlookers, so digest authentication
   # doesn't require SSL. However, this doesn't mean the connection is secure!
   # Just the username and password.
   #
+  # Another common way to authenticate requests is via bearer tokens, a scheme
+  # originally created as part of the OAuth 2.0 protocol (see RFC 6750).
+  #
+  # Bearer authentication sends a token, that can maybe only be a short string
+  # of hexadecimal characters or even a JWT Token. Similarly to the Basic
+  # authentication, this scheme should only be used with SSL.
+  #
   # (You really, really want to use SSL. There's little reason not to.)
   #
   # === Picking an authentication scheme
   #
-  # Basic authentication is the default. To switch to digest authentication,
-  # set +auth_type+ to +:digest+:
+  # Basic authentication is the default. To switch to digest or bearer token authentication,
+  # set +auth_type+ to +:digest+ or +:bearer+:
   #
   #    class Person < ActiveResource::Base
   #      self.auth_type = :digest
@@ -153,6 +163,16 @@ module ActiveResource
   #      self.site = "https://ryan:password@api.people.com"
   #    end
   #
+  # === Setting the bearer token
+  #
+  # Set +bearer_token+ on the class:
+  #
+  #    class Person < ActiveResource::Base
+  #      # Set bearer token directly:
+  #      self.auth_type = :bearer
+  #      self.bearer_token = "my-bearer-token"
+  #    end
+  #
   # === Certificate Authentication
   #
   # You can also authenticate using an X509 certificate. <tt>See ssl_options=</tt> for all options.
@@ -198,7 +218,9 @@ module ActiveResource
   # * 405 - ActiveResource::MethodNotAllowed
   # * 409 - ActiveResource::ResourceConflict
   # * 410 - ActiveResource::ResourceGone
+  # * 412 - ActiveResource::PreconditionFailed
   # * 422 - ActiveResource::ResourceInvalid (rescued by save as validation errors)
+  # * 429 - ActiveResource::TooManyRequests
   # * 401..499 - ActiveResource::ClientError
   # * 500..599 - ActiveResource::ServerError
   # * Other - ActiveResource::ConnectionError
@@ -298,17 +320,24 @@ 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
     class_attribute :include_format_in_path
     self.include_format_in_path = true
 
+    class_attribute :connection_class
+    self.connection_class = Connection
 
     class << self
       include ThreadsafeAttributes
-      threadsafe_attribute :_headers, :_connection, :_user, :_password, :_site, :_proxy
+      threadsafe_attribute :_headers, :_connection, :_user, :_password, :_bearer_token, :_site, :_proxy
 
       # Creates a schema for this resource - setting the attributes that are
       # known prior to fetching an instance from the remote system.
@@ -374,12 +403,12 @@ 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
 
-          schema
+          @schema
         else
           @schema ||= nil
         end
@@ -396,7 +425,7 @@ module ActiveResource
       # example:
       #
       #   class Person < ActiveResource::Base
-      #     schema = {'name' => :string, 'age' => :integer }
+      #     self.schema = {'name' => :string, 'age' => :integer }
       #   end
       #
       # The keys/values can be strings or symbols. They will be converted to
@@ -413,7 +442,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
 
@@ -461,8 +490,8 @@ module ActiveResource
           self._site = nil
         else
           self._site = create_site_uri_from(site)
-          self._user = URI.parser.unescape(_site.user) if _site.user
-          self._password = URI.parser.unescape(_site.password) if _site.password
+          self._user = URI::DEFAULT_PARSER.unescape(_site.user) if _site.user
+          self._password = URI::DEFAULT_PARSER.unescape(_site.password) if _site.password
         end
       end
 
@@ -514,6 +543,22 @@ module ActiveResource
         self._password = password
       end
 
+      # Gets the \bearer_token for REST HTTP authentication.
+      def bearer_token
+        # Not using superclass_delegating_reader. See +site+ for explanation
+        if _bearer_token_defined?
+          _bearer_token
+        elsif superclass != Object && superclass.bearer_token
+          superclass.bearer_token.dup.freeze
+        end
+      end
+
+      # Sets the \bearer_token for REST HTTP authentication.
+      def bearer_token=(bearer_token)
+        self._connection = nil
+        self._bearer_token = bearer_token
+      end
+
       def auth_type
         if defined?(@auth_type)
           @auth_type
@@ -615,7 +660,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.
@@ -632,10 +677,13 @@ module ActiveResource
       # or not (defaults to <tt>false</tt>).
       def connection(refresh = false)
         if _connection_defined? || superclass == Object
-          self._connection = Connection.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
+          _connection.bearer_token = bearer_token if bearer_token
           _connection.auth_type = auth_type if auth_type
           _connection.timeout = timeout if timeout
           _connection.open_timeout = open_timeout if open_timeout
@@ -648,11 +696,10 @@ module ActiveResource
       end
 
       def headers
-        self._headers ||= {}
-        if superclass != Object && superclass.headers
-          self._headers = superclass.headers.merge(_headers)
+        self._headers ||= if superclass != Object
+          InheritingHash.new(superclass.headers)
         else
-          _headers
+          {}
         end
       end
 
@@ -678,15 +725,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)
@@ -701,9 +748,9 @@ 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}" }
+        prefix_call = value.gsub(/:\w+/) { |key| "\#{URI::DEFAULT_PARSER.escape options[#{key}].to_s}" }
 
         # Clear prefix parameters in case they have been cached
         @prefix_parameters = nil
@@ -720,10 +767,10 @@ module ActiveResource
         raise
       end
 
-      alias_method :set_prefix, :prefix=  #:nodoc:
+      alias_method :set_prefix, :prefix=  # :nodoc:
 
-      alias_method :set_element_name, :element_name=  #:nodoc:
-      alias_method :set_collection_name, :collection_name=  #:nodoc:
+      alias_method :set_element_name, :element_name=  # :nodoc:
+      alias_method :set_collection_name, :collection_name=  # :nodoc:
 
       def format_extension
         include_format_in_path ? ".#{format.extension}" : ""
@@ -759,7 +806,37 @@ module ActiveResource
         check_prefix_options(prefix_options)
 
         prefix_options, query_options = split_options(prefix_options) if query_options.nil?
-        "#{prefix(prefix_options)}#{collection_name}/#{URI.parser.escape id.to_s}#{format_extension}#{query_string(query_options)}"
+        "#{prefix(prefix_options)}#{collection_name}/#{URI.encode_www_form_component(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.
@@ -809,7 +886,7 @@ module ActiveResource
         "#{prefix(prefix_options)}#{collection_name}#{format_extension}#{query_string(query_options)}"
       end
 
-      alias_method :set_primary_key, :primary_key=  #:nodoc:
+      alias_method :set_primary_key, :primary_key=  # :nodoc:
 
       # Builds a new, unsaved record using the default values from the remote server so
       # that it can be used with RESTful forms.
@@ -852,6 +929,17 @@ module ActiveResource
         self.new(attributes).tap { |resource| resource.save }
       end
 
+      # Creates a new resource (just like <tt>create</tt>) and makes a request to the
+      # remote service that it be saved, but runs validations and raises
+      # <tt>ActiveResource::ResourceInvalid</tt>, making it equivalent to the following
+      # simultaneous calls:
+      #
+      #   ryan = Person.new(:first => 'ryan')
+      #   ryan.save!
+      def create!(attributes = {})
+        self.new(attributes).tap { |resource| resource.save! }
+      end
+
       # Core method for finding resources. Used similarly to Active Record's +find+ method.
       #
       # ==== Arguments
@@ -916,11 +1004,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
 
@@ -947,7 +1042,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
 
 
@@ -981,7 +1076,7 @@ module ActiveResource
           prefix_options, query_options = split_options(options[:params])
           path = element_path(id, prefix_options, query_options)
           response = connection.head(path, headers)
-          response.code.to_i == 200
+          (200..206).include? response.code.to_i
         end
         # id && !find_single(id, options).nil?
       rescue ActiveResource::ResourceNotFound, ActiveResource::ResourceGone
@@ -989,7 +1084,6 @@ module ActiveResource
       end
 
       private
-
         def check_prefix_options(prefix_options)
           p_options = HashWithIndifferentAccess.new(prefix_options)
           prefix_parameters.each do |p|
@@ -999,23 +1093,26 @@ module ActiveResource
 
         # Find every resource
         def find_every(options)
-          begin
+          params = options[:params]
+          prefix_options, query_options = split_options(params)
+
+          response =
             case from = options[:from]
             when Symbol
-              instantiate_collection(get(from, options[:params]), options[:params])
+              get(from, params)
             when String
-              path = "#{from}#{query_string(options[:params])}"
-              instantiate_collection(format.decode(connection.get(path, headers).body) || [], options[:params])
+              path = "#{from}#{query_string(query_options)}"
+              format.decode(connection.get(path, headers).body)
             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 )
+              format.decode(connection.get(path, headers).body)
             end
-          rescue ActiveResource::ResourceNotFound
-            # Swallowing ResourceNotFound exceptions and return nil - as per
-            # ActiveRecord.
-            nil
-          end
+
+          instantiate_collection(response || [], query_options, prefix_options)
+        rescue ActiveResource::ResourceNotFound
+          # Swallowing ResourceNotFound exceptions and return nil - as per
+          # ActiveRecord.
+          nil
         end
 
         # Find a single resource from a one-off URL
@@ -1076,16 +1173,16 @@ 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 ]
         end
     end
 
-    attr_accessor :attributes #:nodoc:
-    attr_accessor :prefix_options #:nodoc:
+    attr_accessor :attributes # :nodoc:
+    attr_accessor :prefix_options # :nodoc:
 
     # If no schema has been defined for the class (see
     # <tt>ActiveResource::schema=</tt>), the default automatic schema is
@@ -1142,13 +1239,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 = attributes.reject { |k, v| k == self.class.primary_key || v.is_a?(ActiveResource::Base) }.transform_values { |v| 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
 
@@ -1323,13 +1420,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
 
@@ -1345,7 +1442,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
@@ -1370,7 +1467,11 @@ module ActiveResource
     #   your_supplier.load(my_attrs)
     #   your_supplier.save
     def load(attributes, remove_root = false, persisted = false)
-      raise ArgumentError, "expected an attributes Hash, got #{attributes.inspect}" unless attributes.is_a?(Hash)
+      unless attributes.respond_to?(:to_hash)
+        raise ArgumentError, "expected attributes to be able to convert to Hash, got #{attributes.inspect}"
+      end
+
+      attributes = attributes.to_hash
       @prefix_options, attributes = split_options(attributes)
 
       if attributes.keys.size == 1
@@ -1382,21 +1483,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
@@ -1438,7 +1539,7 @@ module ActiveResource
     # A method to determine if an object responds to a message (e.g., a method call). In Active Resource, a Person object with a
     # +name+ attribute can answer <tt>true</tt> to <tt>my_person.respond_to?(:name)</tt>, <tt>my_person.respond_to?(:name=)</tt>, and
     # <tt>my_person.respond_to?(:name?)</tt>.
-    def respond_to?(method, include_priv = false)
+    def respond_to_missing?(method, include_priv = false)
       method_name = method.to_s
       if attributes.nil?
         super
@@ -1453,12 +1554,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))
     end
 
-    def to_xml(options={})
-      super({ :root => self.class.element_name }.merge(options))
+    def read_attribute_for_serialization(n)
+      if !attributes[n].nil?
+        attributes[n]
+      elsif respond_to?(n)
+        send(n)
+      end
     end
 
     protected
@@ -1486,9 +1595,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
@@ -1496,13 +1605,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
@@ -1512,14 +1625,9 @@ module ActiveResource
       end
 
     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
@@ -1532,7 +1640,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]
@@ -1549,7 +1657,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("::")
@@ -1565,11 +1677,20 @@ 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))
+        resource = Class.new(ActiveResource::Base)
         resource.prefix = self.class.prefix
-        resource.site   = self.class.site
+        resource.site = self.class.site
+        self.class.const_set(resource_name, resource)
+
         resource
       end
 
@@ -1577,7 +1698,7 @@ module ActiveResource
         self.class.__send__(:split_options, options)
       end
 
-      def method_missing(method_symbol, *arguments) #:nodoc:
+      def method_missing(method_symbol, *arguments) # :nodoc:
         method_name = method_symbol.to_s
 
         if method_name =~ /(=|\?)$/
@@ -1600,7 +1721,7 @@ module ActiveResource
     extend ActiveModel::Naming
     extend ActiveResource::Associations
 
-    include Callbacks, CustomMethods, Observing, Validations
+    include Callbacks, CustomMethods, Validations
     include ActiveModel::Conversion
     include ActiveModel::Serializers::JSON
     include ActiveModel::Serializers::Xml
@@ -1609,4 +1730,3 @@ module ActiveResource
 
   ActiveSupport.run_load_hooks(:active_resource, Base)
 end
-