From 2de60a926b541d837d7a927bb2efc8135679ac40 Mon Sep 17 00:00:00 2001 From: akihiro17 Date: Thu, 27 Aug 2015 14:05:47 +0900 Subject: Set the content-type to `text/html` if the options[:html] is true In this commit, we set the content-type to `text/html` in AbstractController if the `options[:html]` is true so that we don't include ActionView::Rendering into ActionController::Metal to set it properly. I removed the if `options[:plain]` statement because `AbstractController#rendered_format` returns `Mime::TEXT` by default. --- actionpack/test/controller/new_base/render_html_test.rb | 1 - 1 file changed, 1 deletion(-) (limited to 'actionpack/test/controller/new_base') diff --git a/actionpack/test/controller/new_base/render_html_test.rb b/actionpack/test/controller/new_base/render_html_test.rb index 49c1b67d04..e9ea57e329 100644 --- a/actionpack/test/controller/new_base/render_html_test.rb +++ b/actionpack/test/controller/new_base/render_html_test.rb @@ -4,7 +4,6 @@ module RenderHtml class MinimalController < ActionController::Metal include AbstractController::Rendering include ActionController::Rendering - include ActionView::Rendering def index render html: "Hello World!" -- cgit v1.2.3