Skip to content

Commit 87bc119

Browse files
committed
Remove deprecated constant AbstractController::Helpers::MissingHelperError
1 parent b2f96d0 commit 87bc119

File tree

4 files changed

+6
-24
lines changed

4 files changed

+6
-24
lines changed

actionpack/CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
* Remove deprecated constant `AbstractController::Helpers::MissingHelperError`.
2+
3+
*Rafael Mendonça França*
4+
15
* Fix a race condition that could cause a `Text file busy - chromedriver`
26
error with parallel system tests
37

actionpack/lib/abstract_controller/helpers.rb

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
module AbstractController
77
module Helpers
8-
include ActiveSupport::Deprecation::DeprecatedConstantAccessor
98
extend ActiveSupport::Concern
109

1110
included do
@@ -24,23 +23,6 @@ module Helpers
2423
self._helpers = define_helpers_module(self)
2524
end
2625

27-
class DeprecatedMissingHelperError < LoadError
28-
def initialize(error, path)
29-
@error = error
30-
@path = "helpers/#{path}.rb"
31-
set_backtrace error.backtrace
32-
33-
if /^#{path}(\.rb)?$/.match?(error.path)
34-
super("Missing helper file helpers/%s.rb" % path)
35-
else
36-
raise error
37-
end
38-
end
39-
end
40-
deprecate_constant "MissingHelperError", "AbstractController::Helpers::DeprecatedMissingHelperError",
41-
message: "AbstractController::Helpers::MissingHelperError has been deprecated. If a Helper is not present, a NameError will be raised instead.",
42-
deprecator: AbstractController.deprecator
43-
4426
def _helpers
4527
self.class._helpers
4628
end

actionpack/test/controller/helper_test.rb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -264,12 +264,6 @@ def test_helper_proxy_config
264264
assert_equal "smth", AllHelpersController.helpers.config.my_var
265265
end
266266

267-
def test_missing_helper_error_is_deprecated
268-
assert_deprecated(AbstractController.deprecator) do
269-
AbstractController::Helpers::MissingHelperError
270-
end
271-
end
272-
273267
private
274268
def expected_helper_methods
275269
TestHelper.instance_methods

guides/source/7_2_release_notes.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ Please refer to the [Changelog][action-pack] for detailed changes.
5252

5353
* Remove deprecated constant `ActionDispatch::IllegalStateError`.
5454

55+
* Remove deprecated constant `AbstractController::Helpers::MissingHelperError`.
56+
5557
### Deprecations
5658

5759
### Notable changes

0 commit comments

Comments
 (0)