aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
diff options
context:
space:
mode:
authorCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2012-01-15 15:37:27 -0200
committerCarlos Antonio da Silva <carlosantoniodasilva@gmail.com>2012-01-17 10:04:38 -0200
commit53596d091390bda7fb9b78ad838273a9e63fbde1 (patch)
tree85cb372492de6e8e9191afad2322691969eb8dc3 /actionpack
parent6244d16eaf8eea9ce963f08dced60399b121a6dc (diff)
downloadrails-53596d091390bda7fb9b78ad838273a9e63fbde1.tar.gz
rails-53596d091390bda7fb9b78ad838273a9e63fbde1.tar.bz2
rails-53596d091390bda7fb9b78ad838273a9e63fbde1.zip
Refactor render nothing/text => nil logic, and move to right place
Options :nothing and :text => nil should be handled by ActionController::Rendering instead.
Diffstat (limited to 'actionpack')
-rw-r--r--actionpack/lib/abstract_controller/rendering.rb3
-rw-r--r--actionpack/lib/action_controller/metal/rendering.rb4
2 files changed, 4 insertions, 3 deletions
diff --git a/actionpack/lib/abstract_controller/rendering.rb b/actionpack/lib/abstract_controller/rendering.rb
index bca0dcffd8..ddc93464cd 100644
--- a/actionpack/lib/abstract_controller/rendering.rb
+++ b/actionpack/lib/abstract_controller/rendering.rb
@@ -165,9 +165,6 @@ module AbstractController
options[:prefixes] ||= _prefixes
end
- options[:text] = nil if options.delete(:nothing) == true
- options[:text] = " " if options.key?(:text) && options[:text].nil?
-
options[:template] ||= (options[:action] || action_name).to_s
options
end
diff --git a/actionpack/lib/action_controller/metal/rendering.rb b/actionpack/lib/action_controller/metal/rendering.rb
index 70fd79bb8b..f6e9e4c4ac 100644
--- a/actionpack/lib/action_controller/metal/rendering.rb
+++ b/actionpack/lib/action_controller/metal/rendering.rb
@@ -44,6 +44,10 @@ module ActionController
options[:text] = options[:text].to_text
end
+ if options.delete(:nothing) || (options.key?(:text) && options[:text].nil?)
+ options[:text] = " "
+ end
+
if options[:status]
options[:status] = Rack::Utils.status_code(options[:status])
end