Skip to content

Commit

Permalink
Merge pull request #10130 from Agis-/patch-2
Browse files Browse the repository at this point in the history
Mark unused block parameters.
  • Loading branch information
carlosantoniodasilva committed Apr 8, 2013
2 parents d695100 + 1e784cb commit 0d8a76d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions actionpack/lib/action_controller/test_case.rb
Expand Up @@ -18,7 +18,7 @@ def setup_subscriptions
@_layouts = Hash.new(0)
@_files = Hash.new(0)

ActiveSupport::Notifications.subscribe("render_template.action_view") do |name, start, finish, id, payload|
ActiveSupport::Notifications.subscribe("render_template.action_view") do |_name, _start, _finish, _id, payload|
path = payload[:layout]
if path
@_layouts[path] += 1
Expand All @@ -28,7 +28,7 @@ def setup_subscriptions
end
end

ActiveSupport::Notifications.subscribe("!render_template.action_view") do |name, start, finish, id, payload|
ActiveSupport::Notifications.subscribe("!render_template.action_view") do |_name, _start, _finish, _id, payload|
path = payload[:virtual_path]
next unless path
partial = path =~ /^.*\/_[^\/]*$/
Expand All @@ -41,7 +41,7 @@ def setup_subscriptions
@_templates[path] += 1
end

ActiveSupport::Notifications.subscribe("!render_template.action_view") do |name, start, finish, id, payload|
ActiveSupport::Notifications.subscribe("!render_template.action_view") do |_name, _start, _finish, _id, payload|
next if payload[:virtual_path] # files don't have virtual path

path = payload[:identifier]
Expand Down

0 comments on commit 0d8a76d

Please sign in to comment.