Made caching work for WEBrick and lighttpd by appending .html for all URLs not already containing a dot

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

2 files changed, 7 insertions, 8 deletions

diff --git a/actionpack/lib/action_controller/caching.rb b/actionpack/lib/action_controller/caching.rb
index 996b5af08d..98dd9c9df6 100644
--- a/actionpack/lib/action_controller/caching.rb
+++ b/actionpack/lib/action_controller/caching.rb
@@ -90,11 +90,7 @@ module ActionController #:nodoc:
         
         private
           def page_cache_path(path)
-            if path[-1,1] == '/'
-              page_cache_directory + path + '/index'
-            else
-              page_cache_directory + path
-            end
+            page_cache_directory + path + ".html"
           end
       end
 
@@ -115,13 +111,14 @@ module ActionController #:nodoc:
       # If no options are provided, the current +options+ for this action is used. Example:
       #   cache_page "I'm the cached content", :controller => "lists", :action => "show"
       def cache_page(content = nil, options = {})
+        logger.info "Cached page: #{options.inspect} || #{caching_allowed}"
         return unless perform_caching && caching_allowed
         self.class.cache_page(content || @response.body, url_for(options.merge({ :only_path => true })))
       end
 
       private
         def caching_allowed
-          !@request.post? && (@request.parameters.reject { |k, v| %w( id action controller ).include?(k) }).empty?
+          !@request.post?
         end
     end
 
diff --git a/actionpack/lib/action_controller/components.rb b/actionpack/lib/action_controller/components.rb
index 877d8888bd..6371a4255d 100644
--- a/actionpack/lib/action_controller/components.rb
+++ b/actionpack/lib/action_controller/components.rb
@@ -5,8 +5,9 @@ module ActionController #:nodoc:
       base.helper do
         def render_component(options) 
           @controller.logger.info("Start rendering component (#{options.inspect}): ")
-          @controller.send(:component_response, options).body
+          result = @controller.send(:component_response, options).body
           @controller.logger.info("\n\nEnd of component rendering")
+          return result
         end
       end
     end
@@ -15,8 +16,9 @@ module ActionController #:nodoc:
       def render_component(options = {}) #:doc:
         response = component_response(options)
         logger.info "Rendering component (#{options.inspect}): "
-        render_text(response.body, response.headers["Status"])
+        result = render_text(response.body, response.headers["Status"])
         logger.info("\n\nEnd of component rendering")
+        return result
       end
   
     private