906aebceed
Resolved all the conflicts since 2.3.0 -> HEAD. Following is a list of commits that could not be applied cleanly or are obviated with the abstract_controller refactor. They all need to be revisited to ensure that fixes made in 2.3 do not reappear in 3.0: 2259ecf368e6a6715966f69216e3ee86bf1a82a7 AR not available * This will be reimplemented with ActionORM or equivalent 06182ea02e92afad579998aa80144588e8865ac3 implicitly rendering a js response should not use the default layout [#1844 state:resolved] * This will be handled generically 893e9eb99504705419ad6edac14d00e71cef5f12 Improve view rendering performance in development mode and reinstate template recompiling in production [#1909 state:resolved] * We will need to reimplement rails-dev-boost on top of the refactor; the changes here are very implementation specific and cannot be cleanly applied. The following commits are implicated: 199e750d46c04970b5e7684998d09405648ecbd4 3942cb406e1d5db0ac00e03153809cc8dc4cc4db f8ea9f85d4f1e3e6f3b5d895bef6b013aa4b0690 e3b166aab37ddc2fbab030b146eb61713b91bf55 ae9f258e03c9fd5088da12c1c6cd216cc89a01f7 44423126c6f6133a1d9cf1d0832b527e8711d40f 0cb020b4d6d838025859bd60fb8151c8e21b8e84 workaround for picking layouts based on wrong view_paths [#1974 state:resolved] * The specifics of this commit no longer apply. Since it is a two-line commit, we will reimplement this change. 8c5cc66a831aadb159f3daaffa4208064c30af0e make action_controller/layouts pick templates from the current instance's view_paths instead of the class view_paths [#1974 state:resolved] * This does not apply at all. It should be trivial to apply the feature to the reimplemented ActionController::Base. 87e8b162463f13bd50d27398f020769460a770e3 fix HTML fallback for explicit templates [#2052 state:resolved] * There were a number of patches related to this that simply compounded each other. Basically none of them apply cleanly, and the underlying issue needs to be revisited. After discussing the underlying problem with Koz, we will defer these fixes for further discussion.
130 lines
5.8 KiB
Ruby
130 lines
5.8 KiB
Ruby
module ActionController #:nodoc:
|
|
module Verification #:nodoc:
|
|
def self.included(base) #:nodoc:
|
|
base.extend(ClassMethods)
|
|
end
|
|
|
|
# This module provides a class-level method for specifying that certain
|
|
# actions are guarded against being called without certain prerequisites
|
|
# being met. This is essentially a special kind of before_filter.
|
|
#
|
|
# An action may be guarded against being invoked without certain request
|
|
# parameters being set, or without certain session values existing.
|
|
#
|
|
# When a verification is violated, values may be inserted into the flash, and
|
|
# a specified redirection is triggered. If no specific action is configured,
|
|
# verification failures will by default result in a 400 Bad Request response.
|
|
#
|
|
# Usage:
|
|
#
|
|
# class GlobalController < ActionController::Base
|
|
# # Prevent the #update_settings action from being invoked unless
|
|
# # the 'admin_privileges' request parameter exists. The
|
|
# # settings action will be redirected to in current controller
|
|
# # if verification fails.
|
|
# verify :params => "admin_privileges", :only => :update_post,
|
|
# :redirect_to => { :action => "settings" }
|
|
#
|
|
# # Disallow a post from being updated if there was no information
|
|
# # submitted with the post, and if there is no active post in the
|
|
# # session, and if there is no "note" key in the flash. The route
|
|
# # named category_url will be redirected to if verification fails.
|
|
#
|
|
# verify :params => "post", :session => "post", "flash" => "note",
|
|
# :only => :update_post,
|
|
# :add_flash => { "alert" => "Failed to create your message" },
|
|
# :redirect_to => :category_url
|
|
#
|
|
# Note that these prerequisites are not business rules. They do not examine
|
|
# the content of the session or the parameters. That level of validation should
|
|
# be encapsulated by your domain model or helper methods in the controller.
|
|
module ClassMethods
|
|
# Verify the given actions so that if certain prerequisites are not met,
|
|
# the user is redirected to a different action. The +options+ parameter
|
|
# is a hash consisting of the following key/value pairs:
|
|
#
|
|
# <tt>:params</tt>::
|
|
# a single key or an array of keys that must be in the <tt>params</tt>
|
|
# hash in order for the action(s) to be safely called.
|
|
# <tt>:session</tt>::
|
|
# a single key or an array of keys that must be in the <tt>session</tt>
|
|
# in order for the action(s) to be safely called.
|
|
# <tt>:flash</tt>::
|
|
# a single key or an array of keys that must be in the flash in order
|
|
# for the action(s) to be safely called.
|
|
# <tt>:method</tt>::
|
|
# a single key or an array of keys--any one of which must match the
|
|
# current request method in order for the action(s) to be safely called.
|
|
# (The key should be a symbol: <tt>:get</tt> or <tt>:post</tt>, for
|
|
# example.)
|
|
# <tt>:xhr</tt>::
|
|
# true/false option to ensure that the request is coming from an Ajax
|
|
# call or not.
|
|
# <tt>:add_flash</tt>::
|
|
# a hash of name/value pairs that should be merged into the session's
|
|
# flash if the prerequisites cannot be satisfied.
|
|
# <tt>:add_headers</tt>::
|
|
# a hash of name/value pairs that should be merged into the response's
|
|
# headers hash if the prerequisites cannot be satisfied.
|
|
# <tt>:redirect_to</tt>::
|
|
# the redirection parameters to be used when redirecting if the
|
|
# prerequisites cannot be satisfied. You can redirect either to named
|
|
# route or to the action in some controller.
|
|
# <tt>:render</tt>::
|
|
# the render parameters to be used when the prerequisites cannot be satisfied.
|
|
# <tt>:only</tt>::
|
|
# only apply this verification to the actions specified in the associated
|
|
# array (may also be a single value).
|
|
# <tt>:except</tt>::
|
|
# do not apply this verification to the actions specified in the associated
|
|
# array (may also be a single value).
|
|
def verify(options={})
|
|
before_filter :only => options[:only], :except => options[:except] do |c|
|
|
c.__send__ :verify_action, options
|
|
end
|
|
end
|
|
end
|
|
|
|
private
|
|
|
|
def verify_action(options) #:nodoc:
|
|
if prereqs_invalid?(options)
|
|
flash.update(options[:add_flash]) if options[:add_flash]
|
|
response.headers.merge!(options[:add_headers]) if options[:add_headers]
|
|
apply_remaining_actions(options) unless performed?
|
|
end
|
|
end
|
|
|
|
def prereqs_invalid?(options) # :nodoc:
|
|
verify_presence_of_keys_in_hash_flash_or_params(options) ||
|
|
verify_method(options) ||
|
|
verify_request_xhr_status(options)
|
|
end
|
|
|
|
def verify_presence_of_keys_in_hash_flash_or_params(options) # :nodoc:
|
|
[*options[:params] ].find { |v| params[v].nil? } ||
|
|
[*options[:session]].find { |v| session[v].nil? } ||
|
|
[*options[:flash] ].find { |v| flash[v].nil? }
|
|
end
|
|
|
|
def verify_method(options) # :nodoc:
|
|
[*options[:method]].all? { |v| request.method != v.to_sym } if options[:method]
|
|
end
|
|
|
|
def verify_request_xhr_status(options) # :nodoc:
|
|
request.xhr? != options[:xhr] unless options[:xhr].nil?
|
|
end
|
|
|
|
def apply_redirect_to(redirect_to_option) # :nodoc:
|
|
(redirect_to_option.is_a?(Symbol) && redirect_to_option != :back) ? self.__send__(redirect_to_option) : redirect_to_option
|
|
end
|
|
|
|
def apply_remaining_actions(options) # :nodoc:
|
|
case
|
|
when options[:render] ; render(options[:render])
|
|
when options[:redirect_to] ; redirect_to(apply_redirect_to(options[:redirect_to]))
|
|
else head(:bad_request)
|
|
end
|
|
end
|
|
end
|
|
end |