Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

There is already a Set of non-hidden action_names lying around.

  • Loading branch information...
commit 7cc5bf51b4c16e89340b8c45ca0297e387226aaf 1 parent 872d8c3
@thedarkone thedarkone authored tenderlove committed
Showing with 2 additions and 8 deletions.
  1. +2 −8 actionpack/lib/action_controller/metal/hide_actions.rb
View
10 actionpack/lib/action_controller/metal/hide_actions.rb
@@ -26,20 +26,14 @@ def hide_action(*args)
self.hidden_actions = hidden_actions.dup.merge(args.map(&:to_s)).freeze
end
- def inherited(klass)
- klass.class_eval { @visible_actions = {} }
- super
- end
-
def visible_action?(action_name)
- return @visible_actions[action_name] if @visible_actions.key?(action_name)
- @visible_actions[action_name] = !hidden_actions.include?(action_name)
+ action_methods.include?(action_name)
end
# Overrides AbstractController::Base#action_methods to remove any methods
# that are listed as hidden methods.
def action_methods
- @action_methods ||= Set.new(super.reject { |name| hidden_actions.include?(name) })
+ @action_methods ||= Set.new(super.reject { |name| hidden_actions.include?(name) }).freeze
end
end
end
Please sign in to comment.
Something went wrong with that request. Please try again.