I have a method that does something like this:
before_filter :authenticate_rights, :only => [:show]
def authenticate_rights
project = Project.find(params[:id])
redirect_to signin_path unless project.hidden
end
I also want to use this method in some other Controllers, so i copied the method to a helper that is included in the application_controller.
the problem is, that in some controllers, the id for the project isn't the :id
symbol but f.e. :project_id
(and also a :id
is present (for another model)
How would you solve this problem? is there an option to add a parameter to the before_filter action (to pass the right param)?
I'd do it like this:
before_filter { |c| c.authenticate_rights correct_id_here }
def authenticate_rights(project_id)
project = Project.find(project_id)
redirect_to signin_path unless project.hidden
end
Where correct_id_here
is the relevant id to access a Project
.
With some syntactic sugar:
before_filter -> { find_campaign params[:id] }, only: [:show, :edit, :update, :destroy]
Or if you decide to get even more fancy:
before_filter ->(param=params[:id]) { find_campaign param }, only: %i|show edit update destroy|
And since Rails 4 before_action
, a synonym to before_filter
, was introduced, so it can be written as:
before_action ->(param=params[:id]) { find_campaign param }, only: %i|show edit update destroy|
NB
->
stands for lambda
, called lambda literal, introduce in Ruby 1.9
%i
will create an array of symbols
To continue @alex' answer, if you want to :except
or :only
some methods, here is the syntax:
before_filter :only => [:edit, :update, :destroy] do |c| c.authenticate_rights params[:id] end
Found here.
I find the block method using curly braces instead of do...end
to be the clearest option
before_action(only: [:show]) { authenticate_rights(id) }
before_action
is just the newer preferred syntax for before_filter
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With