Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #12412 from bf4/allow_pathname_for_require_dependency | Xavier Noria | 2013-09-30 | 1 | -0/+11 |
|\ | | | | | Allow Pathname for require dependency | ||||
| * | require_dependency should allow Pathname-like objects, not just String | Benjamin Fleischer | 2013-09-30 | 1 | -0/+11 |
| | | |||||
* | | Ensure all-caps nested consts marked as autoloaded | Simon Coffey | 2013-08-27 | 1 | -0/+8 |
| | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | Don't blindly call blame_file! on exceptions in ↵ | Andrew Kreiling | 2013-06-09 | 1 | -0/+8 |
|/ | | | | | | | | | | | | ActiveSupport::Dependencies::Loadable It is possible under some environments to receive an Exception that is not extended with Blamable (e.g. JRuby). ActiveSupport::Dependencies::Loadable#load_dependency blindly call blame_file! on the exception which throws it's own NoMethodError exception and hides the original Exception. This commit fixes #9521 | ||||
* | remove redundant var | Vipul A M | 2013-05-09 | 1 | -1/+0 |
| | |||||
* | Merge pull request #9681 from vipulnsward/fix_typo_in_module_name | Carlos Antonio da Silva | 2013-03-12 | 1 | -2/+2 |
|\ | | | | | Fix typo in DependenciesTestHelpers module name | ||||
| * | Fix typo in DependenciesTestHelpers module name | Vipul A M | 2013-03-12 | 1 | -2/+2 |
| | | |||||
* | | Cleanup tests for unused variables | Vipul A M | 2013-03-11 | 1 | -1/+1 |
|/ | |||||
* | Remove unnecessary begin..rescue..end, use only rescue | Akira Matsuda | 2013-01-06 | 1 | -6/+4 |
| | |||||
* | silences "possibly useless use of :: in void context" warnings | Xavier Noria | 2012-12-24 | 1 | -7/+3 |
| | | | | | | | | | | | | | The AS utility silence_warnings does not really silence this one, because it is issued at parse-time. It seemed to in some places because the constant was the only expression in the block and therefore it was its return value, that could potentially be used by silence_warnings are return value of the yield call. To bypass the warning we assign to a variable. The chosen variable is "_" because it is special-cased in parse.c not to issue an "assigned but unused variable" warning in turn. | ||||
* | silence warning: possibly useless use of a constant in void context | kennyj | 2012-12-06 | 1 | -3/+7 |
| | |||||
* | Patched Marshal#load to work with constant autoloading ↵ | Uriel Katz | 2012-12-01 | 1 | -19/+2 |
| | | | | (active_support/dependecies.rb) (issue #8167) | ||||
* | prevent Dependencies#remove_const from autoloading parents [fixes #8301] | Xavier Noria | 2012-11-28 | 1 | -0/+10 |
| | |||||
* | let remove_constant still delete Kernel#autoload constants [rounds #8213] | Xavier Noria | 2012-11-15 | 1 | -2/+2 |
| | | | | | The method #remove_const does not load the file, so we can still remove the constant. | ||||
* | dependencies no longer trigger Kernel#autoload in remove_const [fixes #8213] | Xavier Noria | 2012-11-15 | 1 | -0/+10 |
| | |||||
* | Fix some assert_raise calls containing messages in Active Support | Carlos Antonio da Silva | 2012-10-28 | 1 | -1/+2 |
| | |||||
* | detect circular constant autoloading | Xavier Noria | 2012-08-28 | 1 | -0/+6 |
| | | | | | | | Nowadays circular autoloads do not work, but the user gets a NameError that says some constant is undefined. That's puzzling, because he is normally trying to autoload a constant he knows can be autoloaded. With this check we can give a better error message. | ||||
* | Fix ActiveSupport tests that depend on run order | Francesco Rodriguez | 2012-08-23 | 1 | -0/+2 |
| | |||||
* | raise the same exception in order to keep path info | Aaron Patterson | 2012-06-12 | 1 | -0/+13 |
| | | | | | | Ruby 2.0.0 implements LoadError#path, but newly raised load errors will not contain the path information. Replace the error message, copy blame, and rereaise the same exception object | ||||
* | removes the obsolete require_association method from dependencies | Xavier Noria | 2012-06-10 | 1 | -4/+0 |
| | | | | | | | | This is an obsolete method from the very early days, apparently it was used circa 2004 because STI support was not smart enough. This method is not public interface, and we are heading a major version, so removal seems right. | ||||
* | use AS::TestCase as the base class | Aaron Patterson | 2012-01-05 | 1 | -2/+2 |
| | |||||
* | Simplify load and require tests | Sam Umbach | 2011-12-03 | 1 | -4/+4 |
| | | | | - These tests don't use autoloading so there's no need to add anything to autoload_paths | ||||
* | Test that require and load raise LoadError if file not found | Sam Umbach | 2011-12-03 | 1 | -0/+16 |
| | |||||
* | Test return value of ActiveSupport::Dependencies::Loadable#load | Sam Umbach | 2011-12-03 | 1 | -0/+16 |
| | |||||
* | Test return value of ActiveSupport::Dependencies::Loadable#require | Sam Umbach | 2011-12-03 | 1 | -0/+47 |
| | | | | | - Add tests to protect from regressions in require's return value behavior - See a10606c490471d8e1483acb3b31d7f2d51e9ebbe (require needs to return true or false) for the original bug fix | ||||
* | Fix autoload_once_paths when using Pathnames & ruby 1.9 | Jonathan del Strother | 2011-08-24 | 1 | -0/+18 |
| | | | | | | | Under ruby 1.9.2 - "/var/log".starts_with?(Pathname.new("/var")) # => false so setting config.autoload_once_paths with Pathnames would not work | ||||
* | Removing extra requires from the test. Already loaded in abstract_unit. | Arun Agrawal | 2011-07-31 | 1 | -1/+0 |
| | |||||
* | Raise NameError instead of ArgumentError in ActiveSupport::Dependencies | Andrew White | 2011-06-01 | 1 | -1/+1 |
| | | | | | | | | ActiveSupport::Dependencies now raises NameError if it finds an existing constant in load_missing_constant. This better reflects the nature of the error which is usually caused by calling constantize on a nested constant. Closes #1423 | ||||
* | Fix failing tests. | José Valim | 2011-05-22 | 1 | -0/+2 |
| | |||||
* | adding deprecation noticies to deprecated class cache methods | Aaron Patterson | 2011-03-02 | 1 | -4/+4 |
| | |||||
* | adding backwards compat for class cache references. <3<3 | Aaron Patterson | 2011-03-02 | 1 | -3/+3 |
| | |||||
* | refactor Reference to a ClassCache object, fix lazy lookup in Middleware so ↵ | Aaron Patterson | 2011-03-01 | 1 | -3/+3 |
| | | | | that anonymous classes are supported | ||||
* | Add before_remove_const callback to ↵ | Andrew White | 2010-09-01 | 1 | -0/+11 |
| | | | | | | ActiveSupport::Dependencies.remove_unloadable_constants! Signed-off-by: Xavier Noria <fxn@hashref.com> | ||||
* | Sadly, this segv's in 1.8 :( | wycats | 2010-08-17 | 1 | -5/+5 |
| | |||||
* | This is for making sure const_missing is triggered without warnings | Santiago Pastorino | 2010-07-26 | 1 | -4/+6 |
| | |||||
* | Fix a bug where requires inside of autoloads were being added to the ↵ | wycats | 2010-07-26 | 1 | -0/+49 |
| | | | | autoloaded_constants list, causing mayhem. [#5165 state:resolved] | ||||
* | Removes unused vars | Santiago Pastorino | 2010-07-24 | 1 | -2/+0 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Makes restore previosly saved warnings_on_first_load value and avoid warning | Santiago Pastorino | 2010-07-24 | 1 | -0/+1 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | renames load_(once_)paths to autoload_(once_)paths in dependencies and config | Xavier Noria | 2010-06-24 | 1 | -18/+18 |
| | |||||
* | ActiveSupport::Dependencies.constantize shortcut for caching named constant ↵ | Jeremy Kemper | 2010-06-05 | 1 | -0/+6 |
| | | | | lookups | ||||
* | Optimize LookupContext | wycats | 2010-06-04 | 1 | -0/+15 |
| | |||||
* | refactor evals and adds some __FILE__ and __LINE__ | Santiago Pastorino | 2010-05-20 | 1 | -1/+1 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Make dependencies_test pass running standalone [#4215 state:resolved] | Santiago Pastorino and Sebastian Martinez | 2010-03-18 | 1 | -1/+1 |
| | | | | Signed-off-by: wycats <wycats@gmail.com> | ||||
* | Simplify dependencies.rb some. Remove alias of Kernel::Foo to Object::Foo as ↵ | Yehuda Katz | 2010-02-14 | 1 | -11/+6 |
| | | | | it was not used and breaks Ruby semantics. | ||||
* | Since require_dependency cannot be cleared, remove the dependencies | Yehuda Katz | 2010-02-10 | 1 | -0/+5 |
| | |||||
* | removes Module#as_load_path, which is no longer used | Xavier Noria | 2010-02-07 | 1 | -5/+0 |
| | |||||
* | Symbol#to_proc is not needed for Ruby >= 1.8.7 | Xavier Noria | 2009-11-09 | 1 | -1/+0 |
| | |||||
* | Fix failing dependencies test relying on . being in LOAD_PATH | Jeremy Kemper | 2009-08-09 | 1 | -2/+4 |
| | |||||
* | Extract repetitive method | Jeremy Kemper | 2009-08-09 | 1 | -33/+37 |
| | |||||
* | Uses &:foo | Jeremy Kemper | 2009-08-09 | 1 | -0/+1 |
| |