diff options
| author | Simon Coffey <simon@tribesports.com> | 2013-07-26 17:37:00 +0100 |
|---|---|---|
| committer | Simon Coffey <simon@tribesports.com> | 2013-08-27 08:20:33 +0100 |
| commit | b4a96686267c8bb261eca21c02acab9fd5f7ca96 (patch) | |
| tree | a4272b7b2414f1986d82d37c5ac0593e12035f50 /activesupport/test/autoloading_fixtures | |
| parent | 701664b56b69827bfb46a5acfddf81d3a81b5d09 (diff) | |
| download | rails-b4a96686267c8bb261eca21c02acab9fd5f7ca96.tar.gz rails-b4a96686267c8bb261eca21c02acab9fd5f7ca96.tar.bz2 rails-b4a96686267c8bb261eca21c02acab9fd5f7ca96.zip | |
Ensure all-caps nested consts marked as autoloaded
Previously, an autoloaded constant `HTML::SomeClass` would not be marked
as autoloaded by AS::Dependencies. This is because the
`#loadable_constants_for_path` method uses `String#camelize` on the
inferred file path, which in turn means that, unless otherwise directed,
AS::Dependencies watches for loaded constants in the `Html` namespace.
By passing the original qualified constant name to `#load_or_require`,
this inference step is avoided, and the new constant is picked up in the
correct namespace.
Diffstat (limited to 'activesupport/test/autoloading_fixtures')
| -rw-r--r-- | activesupport/test/autoloading_fixtures/html/some_class.rb | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/activesupport/test/autoloading_fixtures/html/some_class.rb b/activesupport/test/autoloading_fixtures/html/some_class.rb new file mode 100644 index 0000000000..b43d15d891 --- /dev/null +++ b/activesupport/test/autoloading_fixtures/html/some_class.rb @@ -0,0 +1,4 @@ +module HTML + class SomeClass + end +end |