Explicitly require active_record/query_cache before using it.

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7419 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

actionpack/CHANGELOG

 *SVN*
 
+* Explicitly require active_record/query_cache before using it.  [Jeremy Kemper]
+
 * Fix layout overriding response status.  #9476 [lotswholetime]
 
 * Add fieldset_tag for generating fieldsets, closes #9477. [djanowski]

actionpack/lib/action_controller/caching.rb

@@ -12,7 +12,10 @@ module ActionController #:nodoc:
   module Caching
     def self.included(base) #:nodoc:
       base.send(:include, Pages, Actions, Fragments)
-      base.send(:include, Sweeping, SqlCache) if defined?(ActiveRecord)
+      if defined?(ActiveRecord)
+        require 'active_record/query_cache'
+        base.send(:include, Sweeping, SqlCache)
+      end
 
       base.class_eval do
         @@perform_caching = true
@@ -656,20 +659,19 @@ def method_missing(method, *arguments)
           end
       end
     end
-    
-    if defined?(ActiveRecord)     
-      module SqlCache
-        def self.included(base) #:nodoc:
+
+    module SqlCache
+      def self.included(base) #:nodoc:
+        if defined?(ActiveRecord) && ActiveRecord::Base.respond_to?(:cache)
           base.alias_method_chain :perform_action, :caching
         end
-        
-        def perform_action_with_caching
-          ActiveRecord::Base.cache do
-            perform_action_without_caching
-          end
+      end
+
+      def perform_action_with_caching
+        ActiveRecord::Base.cache do
+          perform_action_without_caching
         end
       end
     end
-    
   end
 end

activerecord/CHANGELOG

 *SVN*
 
+* Explicitly require active_record/query_cache before using it.  [Jeremy Kemper]
+
 * Fix bug where unserializing an attribute attempts to modify a frozen @attributes hash for a deleted record.  [Rick, marclove]
 
 * Performance: absorb instantiate and initialize_with_callbacks into the Base methods. [Jeremy Kemper]

activerecord/lib/active_record.rb

@@ -75,5 +75,4 @@
 
 require 'active_record/connection_adapters/abstract_adapter'
 
-require 'active_record/query_cache'
 require 'active_record/schema_dumper'

activerecord/lib/active_record/query_cache.rb

@@ -62,7 +62,7 @@ def cache(sql)
         case result
         when Array
           result.collect { |row| row.dup }
-        when Numeric, NilClass, FalseClass
+        when Fixnum, NilClass, FalseClass
           result
         else
           result.dup

activerecord/test/query_cache_test.rb

@@ -4,6 +4,8 @@
 require 'fixtures/task'
 require 'fixtures/course'
 
+require 'active_record/query_cache'
+
 class QueryCacheTest < Test::Unit::TestCase
   fixtures :tasks
   
@@ -117,4 +119,4 @@ def test_new_save
   end
 end
 
-end
+end
\ No newline at end of file