Yes, it would be used in show, edit, update, new, and create. Not needed
for index or destroy. You're email says you would use "expect" but I assume
that was a typo and that you meant "except", as in ":except => [:index,
:destroy]". Should I submit another request with this proposed change ?

Jason


On Mon, Aug 19, 2013 at 12:16 PM, Uģis Ozols <notifications@github.com>wrote:

> In doc/guides/4 - Refinery Extensions/5 - Relating Resources in an
> Extension.textile:
>
> > @@ -225,7 +225,7 @@ module Refinery
> >      module Admin
> >        class EventsController < ::Refinery::AdminController
> >
> > -        before_filter :find_all_event_types, :only => [:show]
>
> In which actions is this used? show, edit, update, new and create? If yes
> then I would rather use expect to limit scope of this finder instead of
> calling it for index and destroy actions too.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/refinery/refinerycms/pull/2385/files#r5851391>
> .
>



-- 
*Jason Greschler
Senior Engineer | DZAP LeaseLabs™

*
The DZAP Group <http://www.dzap.com/> • LeaseLabs<http://www.leaselabs.com/>
*san diego* / san francisco / new york

*LeaseLabs is a 2013 NAHB Pillars Award Finalist*
*
*
CONFIDENTIALITY DISCLAIMER: The information contained in this transmission
is confidential and intended only for the use of the individual or entity
to whom it is addressed. If you are not the intended recipient, you are
hereby notified that any review, disclosure, distribution, or duplication
of this communication, and the information contained in it, is strictly
prohibited. If you have received this communication in error, please notify
the sender immediately and permanently delete the original message,
attachments, and all copies. *
*