Tweak template format rules so that the ACCEPT header is only used if it's text/javascript. This is so ajax actions without a :format param get recognized as Mime::JS. [Rick]

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

actionpack/CHANGELOG

@@ -1,5 +1,7 @@
 *SVN*
 
+* Tweak template format rules so that the ACCEPT header is only used if it's text/javascript.  This is so ajax actions without a :format param get recognized as Mime::JS. [Rick]
+
 * The default respond_to blocks don't set a specific extension anymore, so that both 'show.rjs' and 'show.js.rjs' will work. [Rick]
 
 * Allow layouts with extension of .html.erb.  Closes #8032 [Josh Knowles]

actionpack/lib/action_view/base.rb

@@ -403,7 +403,19 @@ module ActionView #:nodoc:
 
     # symbolized version of the :format parameter of the request, or :html by default.
     def template_format
-      @template_format ||= controller.request.parameters[:format].to_sym rescue :html
+      if @template_format.nil?
+        @template_format =
+          begin
+            if controller.request.accepts.first == Mime::JS
+              :js
+            else
+              controller.request.parameters[:format].to_sym
+            end
+          rescue
+            :html
+          end
+      end
+      @template_format
     end
 
     def template_handler_preferences