correcting the documentation that default to false. it does not default to false. returing either false or nil will not halt the chain unless :terminator is explicitly mentioned

1 files changed, 4 insertions, 5 deletions

diff --git a/activesupport/lib/active_support/callbacks.rb b/activesupport/lib/active_support/callbacks.rb
index 4950d009d1..e179287b24 100644
--- a/activesupport/lib/active_support/callbacks.rb
+++ b/activesupport/lib/active_support/callbacks.rb
@@ -522,14 +522,13 @@ module ActiveSupport
       #
       # 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