Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

renames the :abort deprecation behaviour to :raise

That is a better name, thanks @jeremy.
  • Loading branch information...
commit c7ac0a5b28e1403371e4c65666ec92dd2bf0e0d6 1 parent 4c1454d
Xavier Noria fxn authored
4 activesupport/CHANGELOG.md
View
@@ -1,7 +1,7 @@
-* Adds a new deprecation behaviour that aborts the application. Throwing this
+* Adds a new deprecation behaviour that raises an exception. Throwing this
line into +config/environments/development.rb+
- ActiveSupport::Deprecation.behavior = :abort
+ ActiveSupport::Deprecation.behavior = :raise
will cause the application to raise an +ActiveSupport::DeprecationException+
on deprecations.
4 activesupport/lib/active_support/deprecation/behaviors.rb
View
@@ -7,7 +7,7 @@ class DeprecationException < StandardError
class Deprecation
# Default warning behaviors per Rails.env.
DEFAULT_BEHAVIORS = {
- abort: ->(message, callstack) {
+ raise: ->(message, callstack) {
e = DeprecationException.new(message)
e.set_backtrace(callstack)
raise e
@@ -52,7 +52,7 @@ def behavior
#
# Available behaviors:
#
- # [+abort+] Raises <tt>ActiveSupport::DeprecationException</tt>.
+ # [+raise+] Raise <tt>ActiveSupport::DeprecationException</tt>.
# [+stderr+] Log all deprecation warnings to +$stderr+.
# [+log+] Log all deprecation warnings to +Rails.logger+.
# [+notify+] Use +ActiveSupport::Notifications+ to notify +deprecation.rails+.
6 activesupport/test/deprecation_test.rb
View
@@ -98,8 +98,8 @@ def test_several_behaviors
assert_match(/foo=nil/, @b)
end
- def test_abort_behaviour
- ActiveSupport::Deprecation.behavior = :abort
+ def test_raise_behaviour
+ ActiveSupport::Deprecation.behavior = :raise
message = 'Revise this deprecated stuff now!'
callstack = %w(foo bar baz)
@@ -110,7 +110,7 @@ def test_abort_behaviour
assert_equal message, e.message
assert_equal callstack, e.backtrace
else
- flunk 'the :abort deprecation behaviour should raise the expected exception'
+ flunk 'the :raise deprecation behaviour should raise the expected exception'
end
end
Please sign in to comment.
Something went wrong with that request. Please try again.