diff options
author | Keenan Brock <keenan@thebrocks.net> | 2019-01-07 14:13:45 -0500 |
---|---|---|
committer | Keenan Brock <keenan@thebrocks.net> | 2019-01-09 14:52:46 -0500 |
commit | a930f5ba0654e8c8c7ac7b3febbb573d03559a40 (patch) | |
tree | eb61061ebddf53c0f41b40b04a563cc9bc664121 /actionpack/test/lib/controller | |
parent | 9f1a07af0499080c9fd8815705a03a4c7e8fb506 (diff) | |
download | rails-a930f5ba0654e8c8c7ac7b3febbb573d03559a40.tar.gz rails-a930f5ba0654e8c8c7ac7b3febbb573d03559a40.tar.bz2 rails-a930f5ba0654e8c8c7ac7b3febbb573d03559a40.zip |
Fix safe_constantize to not raise a LoadError.
### Summary
There was an issues when using `safe_constantize` on a string that has
the wrong case.
File `em.rb` defines `EM`.
`"Em".safe_constantize` causes a little confusion with the autoloader.
The autoloader finds file "em.rb",
expecting it to define `Em`, but `Em` is not defined.
The autoloader raises a `LoadError`, which is good,
But `safe_constantize` is defined to return `nil` when a class is not found.
### Before
```
"Em".safe_constantize
LoadError: Unable to autoload constant Em, \
expected rails/activesupport/test/autoloading_fixtures/em.rb to define it
```
### After
```
"Em".safe_constantize
# => nil
```
Diffstat (limited to 'actionpack/test/lib/controller')
0 files changed, 0 insertions, 0 deletions