Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | clean up some warnings on trunk ruby | Aaron Patterson | 2013-10-31 | 1 | -1/+1 |
| | |||||
* | this should be private, no other instances of Root should call it | Aaron Patterson | 2013-10-22 | 1 | -1/+1 |
| | |||||
* | do not search through the keys array twice | Aaron Patterson | 2013-10-22 | 1 | -2/+3 |
| | |||||
* | use the provided block to filter lists | Aaron Patterson | 2013-10-22 | 1 | -11/+5 |
| | |||||
* | stop using `send` so that method privacy is respected and we get a small | Aaron Patterson | 2013-10-22 | 1 | -7/+7 |
| | | | | perf increase | ||||
* | Revert "Deprecate the `eager_load_paths` configuration" | Andrew White | 2013-02-19 | 1 | -32/+12 |
| | | | | | | | | | | | Because of the possibility of lib being unintentionally eager loaded it's been agreed that we'll leave autoload paths and eager load paths separate for Rails 4.0. This reverts commit 0757b3388ffe4f44b60de950d40e18ef05055931. Conflicts: railties/CHANGELOG.md | ||||
* | Deprecate the `eager_load_paths` configuration | Andrew White | 2013-01-24 | 1 | -12/+32 |
| | | | | | | | | | | Since the default in Rails 4.0 is to run in 'threadsafe' mode we need to eager load all of the paths in `autoload_paths` so we alias `eager_load_paths` to it. This may have unintended consequences if you have added 'lib' to `autoload_paths` such as loading unneeded code or code intended only for development and/or test environments. If this applies to your application you should thoroughly check what is being eager loaded. | ||||
* | use multiline do-end blocks and rearrange definition for clarity | Matt Bridges | 2013-01-09 | 1 | -5/+5 |
| | |||||
* | replace #flatten with Array() | Gosha Arinich | 2013-01-02 | 1 | -2/+2 |
| | |||||
* | Revert "Merge pull request #7587 from elia/fix-too-eager-loading" | Rafael Mendonça França | 2012-12-18 | 1 | -12/+6 |
| | | | | | | | | | | This reverts commit 3663057518eb9acf9b1e72f47dcb07038e6b7368. REASON: This caused a regression that add app folder in the eager load path. See #8146 for more information. Conflicts: railties/CHANGELOG.md | ||||
* | Merge pull request #7587 from elia/fix-too-eager-loading | Rafael Mendonça França | 2012-10-29 | 1 | -6/+12 |
| | | | | | | Should not eager_load app/assets Conflicts: railties/CHANGELOG.md | ||||
* | Use Ruby 1.9 Hash syntax in railties | Robin Dupret | 2012-10-14 | 1 | -3/+3 |
| | |||||
* | Remove unneeded attr_reader | Rafael Mendonça França | 2012-10-07 | 1 | -1/+0 |
| | |||||
* | Revert "Use flat_map { } instead of map {}.flatten" | Santiago Pastorino | 2012-10-05 | 1 | -1/+1 |
| | | | | | | | | | | | This reverts commit abf8de85519141496a6773310964ec03f6106f3f. We should take a deeper look to those cases flat_map doesn't do deep flattening. irb(main):002:0> [[[1,3], [1,2]]].map{|i| i}.flatten => [1, 3, 1, 2] irb(main):003:0> [[[1,3], [1,2]]].flat_map{|i| i} => [[1, 3], [1, 2]] | ||||
* | Use flat_map { } instead of map {}.flatten | Santiago Pastorino | 2012-10-05 | 1 | -1/+1 |
| | |||||
* | Merge pull request #6910 from mulder/fix_path_glob | José Valim | 2012-07-01 | 1 | -1/+2 |
|\ | | | | | Persist glob when replacing a path | ||||
| * | Persist glob when replacing a path | Nicholas Mulder | 2012-06-30 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When Rails::Paths::Root's []= is used to replace a path it should persist the previous path's glob. Without passing the glob along we get gnarly bugs when trying to wire up things like engines. module FooEngine class Engine < ::Rails::Engine isolate_namespace FooEngine config.paths['config/initializers'] = "lib/foo_engine/initializers" end end ## Example of behaviour before this commit. # # Before the initializer override: >> FooEngine::Engine.config.paths["config/initializers"].glob => "**/*.rb" # After the initializer override: >> FooEngine::Engine.config.paths["config/initializers"].glob => nil ## Example of behaviour after this commit. # # Before the initializer override: >> FooEngine::Engine.config.paths["config/initializers"].glob => "**/*.rb" # After the initializer override: >> FooEngine::Engine.config.paths["config/initializers"].glob => "**/*.rb" | ||||
* | | Revert "Allow loading external route files from the router" | José Valim | 2012-06-29 | 1 | -9/+1 |
|/ | | | | | | | | | | | | | | This reverts commit 6acebb38bc0637bc05c19d87f8767f16ce79189b. Usage of this feature did not reveal any improvement in existing apps. Conflicts: actionpack/lib/action_dispatch/routing/mapper.rb guides/source/routing.textile railties/lib/rails/engine.rb railties/lib/rails/paths.rb railties/test/paths_test.rb | ||||
* | removing more pathnameisms | Aaron Patterson | 2012-05-23 | 1 | -3/+1 |
| | |||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-04-28 | 1 | -1/+1 |
|\ | |||||
| * | Use <tt>Foo::Bar</tt> instead of +Foo::Bar+ | Mark Rushakof | 2012-04-27 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | The latter doesn't render as code in HTML output. Regex used in Rubymine to locate the latter form: (\+)(:*\w+:(?::|\w)+)(\+) | ||||
* | | Allow loading external route files from the router | Jose and Yehuda | 2012-04-25 | 1 | -1/+11 |
|/ | | | | | | | | | This feature enables the ability to load an external routes file from the router via: draw :filename External routes files go in +config/routes+. This feature works in both engines and applications. | ||||
* | warning removed: `&' interpreted as argument prefix | Arun Agrawal | 2012-03-31 | 1 | -1/+1 |
| | |||||
* | Define Paths::Root#[]= in terms of #add. | Ben Woosley | 2012-03-29 | 1 | -2/+1 |
| | |||||
* | Drop Paths::Root initializer check of #path as it isn't checked in the ↵ | Ben Woosley | 2012-03-29 | 1 | -1/+0 |
| | | | | #path= or anywhere else | ||||
* | Drop unnecessary require in Rails::Paths | Ben Woosley | 2012-03-29 | 1 | -2/+0 |
| | |||||
* | favor composition over inheritance | Aaron Patterson | 2012-03-26 | 1 | -5/+20 |
| | |||||
* | favor composition over inheritance | Aaron Patterson | 2012-03-26 | 1 | -7/+29 |
| | |||||
* | still need to ensure the path is a directory | Aaron Patterson | 2012-03-26 | 1 | -1/+1 |
| | |||||
* | eliminating a branch we do not need. thanks @jeremy | Aaron Patterson | 2012-03-26 | 1 | -30/+3 |
| | |||||
* | stop using *args in order to simplify our constructor | Aaron Patterson | 2012-03-26 | 1 | -5/+4 |
| | |||||
* | Don't depend on ivars. Thanks @fesplugas | Aaron Patterson | 2012-03-26 | 1 | -1/+1 |
| | |||||
* | chdir before globbing so that we don't need to escape directory names. | Aaron Patterson | 2012-03-23 | 1 | -1/+30 |
| | | | | fixes #5521 | ||||
* | use File.directory? as Dir.exists? is only 1.9.2+ | Aaron Patterson | 2011-08-01 | 1 | -2/+2 |
| | |||||
* | Merge pull request #2324 from zenapsis/3-1-stable | Santiago Pastorino | 2011-08-01 | 1 | -0/+4 |
| | | | | Rails 3.1 throws a Errno::ENOTDIR if files are put in assets directories | ||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-07-23 | 1 | -9/+9 |
|\ | |||||
| * | little details seen while doing a pass through what's new in docrails | Xavier Noria | 2011-07-23 | 1 | -8/+8 |
| | | |||||
| * | grammar correction for rails/paths #add method arguments | Matt Jankowski | 2011-07-12 | 1 | -2/+2 |
| | | |||||
* | | Ensure Dir.glob is sorted | Joshua Peek | 2011-07-11 | 1 | -1/+1 |
|/ | |||||
* | fix Rails::Paths::Root document | Jan Xie | 2011-05-25 | 1 | -4/+3 |
| | |||||
* | Removing deprecated config.paths.app.controller API | Guillermo Iguaran | 2011-05-24 | 1 | -26/+0 |
| | |||||
* | commented meta code | Vijay Dev | 2011-05-04 | 1 | -11/+11 |
| | |||||
* | formatting results in comments | Vijay Dev | 2011-05-04 | 1 | -8/+8 |
| | |||||
* | Initialize these options as false if they are not given | Santiago Pastorino | 2011-01-09 | 1 | -4/+4 |
| | |||||
* | Fixes typo | Rafael Mendonça França | 2010-10-06 | 1 | -2/+2 |
| | |||||
* | Provide a cleaner syntax for paths configuration that does not rely on ↵ | José Valim | 2010-10-06 | 1 | -49/+103 |
| | | | | method_missing. | ||||
* | Add skip_eager_load!, skip_autoload! and friends to path objects. | José Valim | 2010-07-17 | 1 | -28/+12 |
| | |||||
* | A few changes were done in this commit: | José Valim | 2010-06-28 | 1 | -20/+31 |
| | | | | | | * Added :autoload to engines path API and redefine usage to be in sync with 6f83a5036d8a9c3f8ed7; * Do not autoload code in *lib* for applications (now you need to explicitly require them). This makes an application behave closer to an engine (code in lib is still autoloaded for plugins); * Always autoload code in app/ for engines and plugins. This makes engines behave closer to an application and should allow us to get rid of the unloadable hack required when controllers inside engines inherit from ApplicationController; | ||||
* | Remove app/views from the load paths [#4226 state:resolved] | José Valim | 2010-04-05 | 1 | -5/+17 |
| | |||||
* | Solve some pendencies. | José Valim | 2010-01-24 | 1 | -9/+14 |
| |