Permalink
Browse files

correcting the documentation that default to false. it does not defau…

…lt to false. returing either false or nil will not halt the chain unless :terminator is explicitly mentioned
  • Loading branch information...
1 parent 0257239 commit cd87cf771acb90c43cd56e5f038fc345a69790f9 Neeraj Singh committed Aug 5, 2010
Showing with 4 additions and 5 deletions.
  1. +4 −5 activesupport/lib/active_support/callbacks.rb
@@ -522,14 +522,13 @@ def reset_callbacks(symbol)
#
# This macro accepts the following options:
#
- # * <tt>:terminator</tt> - Indicates when a before filter is considered
- # to be halted.
+ # * <tt>:terminator</tt> - Indicates when a before filter is considered to be halted.
#
# define_callbacks :validate, :terminator => "result == false"
#
- # In the example above, if any before validate callbacks returns +false+,
- # other callbacks are not executed. Defaults to "false", meaning no value
- # halts the chain.
+ # In the example above, if any before validate callback returns +false+,
+ # other callbacks are not executed. Note that in this case if the callback
+ # returns +nil+ then other callbacks are still executed.
#
# * <tt>:rescuable</tt> - By default, after filters are not executed if
# the given block or a before filter raises an error. Set this option to

0 comments on commit cd87cf7

Please sign in to comment.