aboutsummaryrefslogtreecommitdiffstats
path: root/railties/lib
Commit message (Collapse)AuthorAgeFilesLines
* Remove config.autoload_paths from config/application.rbAndrew White2013-02-191-3/+0
| | | | | | | | | | Most application developers shouldn't be adding paths to `config.autoload_paths` as it won't be eagerly loaded so remove it from the generated config/application.rb file. If a developer does want to autoload from paths outside of the app directory then they should add the path to `config.eager_load_paths` as these paths are also autoloaded as well.
* Don't unhook autoloading in productionAndrew White2013-02-191-7/+0
| | | | | | | | | | | | | | | Whilst autoloading is known to be not threadsafe, leaving it in place is a softer solution than failing hard when an application is deployed. Many older applications will have paths added to `autoload_paths` and ideally these should be eagerly loaded to be threadsafe. However one of these paths is quite often lib which could lead to unintended consequences due to the 'junk drawer' nature of this directory. Developers should refrain from adding paths to `autoload_paths` or `eager_load_paths` and use custom folders inside app for code that needs to be eagerly loaded and use `require` or `require_dependency` to explicitly load code from other locations.
* Revert "Deprecate the `eager_load_paths` configuration"Andrew White2013-02-193-52/+28
| | | | | | | | | | | 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
* Added extra comment when a migration already exists of the same name.wangjohn2013-02-151-1/+1
| | | | | | User should know that he can overwrite the previous migration. This comes in handy especially when generating models which were previously created then removed.
* Prefer Ruby 2.0; require 1.9.3+Jeremy Kemper2013-02-091-2/+2
|
* Added notice message for destroy method in scaffoldRahul P. Chaudhari2013-02-101-1/+1
|
* Fix article for generator nameRyunosuke SATO2013-02-071-2/+2
| | | | Use 'some' instead of 'a/an'.
* Added test:all and test:all:db tasks to speed up full test runs.Ryan Davis2013-02-041-0/+12
| | | | | | | | | | Speed up depends on the number of test phases (N) you have for your app, but it is roughly a 1/N improvement. In my app, it goes from 15.51s to 5.56s. Inspired by http://ngauthier.com/2012/02/quick-tests-with-bash.html Reviewed by @tenderlove
* Merge pull request #9175 from robertomiranda/add--rc-descGuillermo Iguaran2013-02-041-0/+3
|\ | | | | Add --rc description on the command line
| * Add --rc description on the command linerobertomiranda2013-02-041-0/+3
| |
* | Use --rc= instead of --rcrobertomiranda2013-02-041-2/+6
|/
* Add --no-rc description on the command linerobertomiranda2013-02-021-0/+3
|
* Refactor --rc optionGuillermo Iguaran2013-02-021-6/+1
|
* Add --rc option to support the load of a custom rc fileAmparo Luna2013-02-011-1/+7
|
* Prefer File.read over File.open.readAkira Matsuda2013-02-011-1/+1
|
* Add --no-rc option to skip the loading of railsrc fileAmparo Luna2013-01-311-6/+8
|
* Remove BestStandardsSupport middlewareGuillermo Iguaran2013-01-293-9/+2
|
* Merge pull request #8303 from jeremywrowe/masterRafael Mendonça França2013-01-251-1/+1
|\ | | | | | | | | | | | | Fixes improper database name when creating a new rails app with a '.' Conflicts: railties/CHANGELOG.md
| * Fixes improper database name when a '.' is included while creating a new railsJeremy W. Rowe2013-01-161-1/+1
| | | | | | | | | | | | application. EG: `rails new something.awesome.com` Conflicts: railties/CHANGELOG.md
* | Deprecate the `eager_load_paths` configurationAndrew White2013-01-243-28/+52
| | | | | | | | | | | | | | | | | | | | 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.
* | Revert "Merge pull request #7873 from steveklabnik/update_initializer"Carlos Antonio da Silva2013-01-222-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 43e14f8ff203c21646f733d35bdde4e68da70a3b, reversing changes made to bb17a0f085aba40a1952612f6c98b2a5804a1152. Conflicts: railties/lib/rails/generators/rails/app/templates/config/application.rb railties/lib/rails/generators/rails/app/templates/config/initializers/locale.rb Reason: setting the Time.zone in an after_initialize block can become a problem for people upgrading, since it's not going to be available when config/initializers run, and people might be relying on it. An example that I noticed was with the Chronic gem, for date/time parsing. It requires us to configure the time class to be Time.zone, and doing that in an initializer no longer worked with these changes, so reverting is the safer path for now.
* | Remove unnecessary requireRyunosuke SATO2013-01-221-1/+0
| | | | | | | | `require 'rubygems'` is already required in Ruby 1.9 or later.
* | Merge pull request #8557 from timraymond/add_back_engines_and_deprecateRafael Mendonça França2013-01-223-5/+34
|\ \ | | | | | | | | | | | | | | | | | | Adding Back Rails::Engine::Railties#engines Conflicts: railties/CHANGELOG.md
| * | Adding Back Rails::Engine::Railties#enginesTim Raymond2013-01-033-5/+34
| | | | | | | | | | | | | | | Removing it breaks functionality with gems such as Thinking Sphinx. This restores it with a deprecation warning. Closes #8551
* | | use tap with block parameterAkira Matsuda2013-01-181-1/+1
| | |
* | | clear specific logs when using rake log:clearMatt Bridges2013-01-161-4/+18
| |/ |/|
* | allow :dirs option for .enumerateBrian D. Burns2013-01-151-6/+15
| | | | | | | | | | | | Allows custom rake tasks to be defined using: SourceAnnotationExtractor.enumerate 'TODO|FIXME', dirs: %w(app lib), tag: true
* | use case statementBrian D. Burns2013-01-151-10/+16
| |
* | Deprecate `Rails::Generators::ActiveModel#update_attributes`Carlos Antonio da Silva2013-01-121-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deprecate #update_attributes in favor of `#update`. ORMs that implement `Generators::ActiveModel#update_attributes` should change to `#update`. Scaffold controller generators should change calls like: @orm_instance.update_attributes(...) to: @orm_instance.update(...) This goes along with the addition of `ActiveRecord::Base#update`, introduced in cb2bd4aa619d9329c42aaf6d9f8eacc616ce53f4.
* | Merge pull request #8887 from rwz/controller-scaffold-codestyleRafael Mendonça França2013-01-102-3/+3
|\ \ | | | | | | Controller scaffold codestyle
| * | More consistent quotes in scaffold generatorPavel Pravosud2013-01-111-2/+2
| | |
| * | Changed if ! to unlessPavel Pravosud2013-01-111-1/+1
| | |
* | | Specified jbuilder versionPavel Pravosud2013-01-111-1/+1
|/ /
* | Include jbuilder by default and rely on its scaffold generator to show json ↵David Heinemeier Hansson2013-01-104-75/+13
| | | | | | | | API. The default scaffold will now just create HTML if the user actively opts out of jbuilder
* | remove performance tests from the default stackYves Senn2013-01-105-13/+2
| |
* | extract PerformanceTest into rails-performance_tests gemYves Senn2013-01-109-134/+0
| |
* | fire a notification when the request stops / startsAaron Patterson2013-01-091-2/+17
| |
* | extract logger and development checks to methodsAaron Patterson2013-01-091-6/+16
| |
* | use multiline do-end blocks and rearrange definition for clarityMatt Bridges2013-01-091-5/+5
| |
* | mv README README.rdoc as app template fileXavier Noria2013-01-082-4/+4
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-01-091-1/+1
|\ \ | | | | | | | | | | | | Conflicts: guides/source/getting_started.md
| * | make symbol reference in docs appear as codeGosha Arinich2013-01-031-1/+1
| | |
* | | Use absolute pathRyunosuke SATO2013-01-081-3/+3
| | | | | | | | | | | | It is semantically correct than relative path.
* | | Don't configure Turn output, we aren't using it since 3.2.0Guillermo Iguaran2013-01-071-7/+0
| | |
* | | Merge pull request #8801 from ktheory/dont_require_turnGuillermo Iguaran2013-01-071-5/+2
|\ \ \ | | | | | | | | Don't require 'turn' from rails/test_help
| * | | Don't require 'turn' from rails/test_helpAaron Suggs2013-01-071-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If 'turn' is in the Gemfile with `:require => nil`, Rails should respect that. This supports the case where 'turn' may or may not be required depending on different developers' preference or environment. E.g., you require turn for local testing, but not on your CI server by putting this in `test/test_helper.rb`: require 'turn' if ENV['TURN']
* | | | Initialize instance variable to remove warningCarlos Antonio da Silva2013-01-071-0/+4
| | | | | | | | | | | | | | | | actions.rb:34: warning: instance variable @in_group not initialized
* | | | new README generated for new appsXavier Noria2013-01-081-252/+16
| | | |
* | | | doc/README_FOR_APP is no longer generatedXavier Noria2013-01-083-12/+5
| | | |
* | | | Revert some warning removals related to Ruby 2.0Carlos Antonio da Silva2013-01-071-2/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These warnings were actually a bug in Ruby 2.0, the accessors should not be raising such warnings, they are only meant for ivars. - Revert "fix warnings in Ruby 2.0" This reverts commit 26702a6d3461f4a1c75165030b96886514ecb877. - Revert "Merge pull request #8282 from arunagw/warning_removed_for_ruby2" This reverts commit f63d6544e45e78cda29c0c56fbdf3d9e1f405340, reversing changes made to 3a890681fad8218305585036abed6d7463a44e41.