aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* added more accurate test for add_on_emptyAngelo capilleri2013-02-191-0/+7
| | | | | with the current tests, if delete the assignment of is_empty in add_on_empty method the tests not fail. With this test, if we delete is_empty, the test fails
* moves a CHANGELOG entry from AR to APXavier Noria2013-02-192-4/+4
|
* small tweaks in the AR CHANGELOGXavier Noria2013-02-191-4/+4
|
* Merge pull request #9258 from wangjohn/blank_argument_errors_in_arelRafael Mendonça França2013-02-192-9/+34
|\ | | | | Raising an error when query methods have blank arguments.
| * Raise an error when query methods have blank arguments, and when blank ↵John J Wang2013-02-192-9/+34
|/ | | | arguments are meaningless.
* InvalidMessage is in ActiveSupport::MessageEncryptor namespaceSantiago Pastorino2013-02-192-4/+19
| | | | Closes #9302
* Fix rake routes output in railties testCarlos Antonio da Silva2013-02-191-2/+4
|
* Fix AP tests related to routes inspector output and the new column titlesCarlos Antonio da Silva2013-02-191-20/+65
|
* Add missing require to routes inspectorCarlos Antonio da Silva2013-02-191-0/+1
|
* Merge pull request #9321 from mfilej/action-dispatch-http-url-requiresXavier Noria2013-02-191-0/+3
|\ | | | | Require ActiveSupport dependencies in ActionDispatch::Http::URL
| * Require AS deps in ActionDispatch::Http::URLMiha Filej2013-02-191-0/+3
| | | | | | | | Allows us to use url_for and friends in isolation.
* | 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-197-91/+66
|/ | | | | | | | | | | 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
* Duck typing is hard. Add header to HtmlTableFormatter.Steve Klabnik2013-02-181-0/+4
| | | | | | | Somehow I missed this method, which was then throwing an error when viewing routes as HTML. Make @rubys happy: ✔
* editorial tweaksXavier Noria2013-02-192-4/+7
|
* Use strip_heredoc in the tests as well.Steve Klabnik2013-02-181-5/+6
| | | | Thanks @sikachu. :sweat_smile:
* use strip_heredoc to keep indentation consistent.Steve Klabnik2013-02-181-12/+12
| | | | Thanks @sikachu. :heart:
* Merge pull request #9316 from steveklabnik/no_routesXavier Noria2013-02-183-0/+46
|\ | | | | Print message on stderr when no routes are defined.
| * Add message when you have no routes defined.Steve Klabnik2013-02-183-0/+46
|/ | | | | | Print a message in both `rake routes` and at GET "/rails/info/routes" that lets you know you have no routes defined, as well as linking to the Rails Guide on the topic.
* Merge pull request #9318 from steveklabnik/router_headingsXavier Noria2013-02-181-3/+19
|\ | | | | Add headings to rake routes table
| * Add headings to rake routes tableSteve Klabnik2013-02-181-3/+19
| |
* | Merge pull request #9317 from balexand/patch-1Rafael Mendonça França2013-02-181-0/+1
|\ \ | |/ |/| Document changes from #7968 in Upgrading Guide
| * Document changes in #7968Brian Alexander2013-02-181-0/+1
|/
* Bump up minitest dependency and remove our own reinvented version of ↵Akira Matsuda2013-02-192-38/+2
| | | | ParallelEach
* Gist URLs are now namespacedAkira Matsuda2013-02-186-11/+11
| | | | see: https://github.com/blog/1406-namespaced-gists
* Merge pull request #9313 from bkzl/return-false-on-toggle-on-error-pagesGuillermo Iguaran2013-02-172-6/+7
|\ | | | | Return false in toggle function on error pages
| * Return false on toggle on error pagesBartlomiej Kozal2013-02-172-6/+7
| |
* | Merge pull request #9312 from Takehiro-Adachi/add_tests_for_ar_createGuillermo Iguaran2013-02-171-0/+8
|\ \ | |/ |/| Add test for ActiveRecord::Base#create
| * add tests for ActiveRecord::Base#createTakehiro Adachi2013-02-181-0/+8
|/
* Add CHANGELOG entry for 41ff6a10216f48f43605a1f9cd6094765cab750fAndrew White2013-02-171-0/+40
| | | | [ci skip]
* Merge pull request #9307 from wangjohn/remove_unneeded_md5_digestXavier Noria2013-02-161-1/+0
|\ | | | | Removed an old require of an MD5 digest which is no longer used.
| * Removed an old require of an MD5 digest which is no longer used.wangjohn2013-02-151-1/+0
| |
* | copy-edits RDoc in fba496fXavier Noria2013-02-161-4/+4
| | | | | | | | | | | | | | This revision makes the terminology uniform with the rest of the file. In general we describe these methods in terms of the adapter, rather than the database. (As a side note, if we wanted to refer to the database we would write "PostgreSQL".)
* | Remove valid_type? methodJon Leighton2013-02-154-17/+0
| | | | | | | | | | It was supposed to be removed by the preceding two reversions but I did them in the wrong order.
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2013-02-1511-9/+68
|\ \ | | | | | | | | | | | | Conflicts: guides/source/upgrading_ruby_on_rails.md
| * | copy edits [ci skip]Vijay Dev2013-02-152-3/+3
| | |
| * | Revert "incorrect content of models/comment.rb after 'rails generate'"Rafael Mendonça França2013-02-131-2/+0
| | | | | | | | | | | | | | | | | | | | | This reverts commit 0f0803fda5df60319d8646bb4f3e71f2e596316a. Reason: The content was correct since attr_accessible was removed from rails master.
| * | incorrect content of models/comment.rb after 'rails generate'Kuba Brecka2013-02-131-0/+2
| | |
| * | Correct bullet content line wrappingTim "Roger" Harvey2013-02-121-1/+1
| | | | | | | | | | | | | | | | | | The `:prefix` bullet content was not properly wrapping, dropping to the following line because the additional content was not indented properly. With this change, it will now display properly the way `:discard_type` does.
| * | Detail Ruby 1.8 ships with TestUnit and Ruby 1.9 with MiniTestAmerico Savinon2013-02-121-1/+4
| | |
| * | Fix typo.Carson McDonald2013-02-111-1/+1
| | | | | | | | | | | | [ci skip]
| * | Fix supported ruby version for Rails 4Ryunosuke SATO2013-02-101-1/+1
| | |
| * | Fix CoffeeScript exampleRyunosuke SATO2013-02-101-1/+1
| | | | | | | | | | | | `a[data-color]` is not used in example html.
| * | Clarify ActiveModel::Validator initialization in docs.Henrik Nyh2013-02-091-0/+3
| | | | | | | | | As discussed with @josevalim on Ruby Rogues Parley.
| * | Validations guide: validates_with init clarification.Henrik Nyh2013-02-091-0/+29
| | | | | | | | | As discussed with @josevalim on Ruby Rogues Parley.
| * | Add example for ActiveRecord::Base.joins using a string as argumentMaurizio De Santis2013-02-071-0/+5
| | |
| * | Add example for ActiveRecord::Base.group using a string as argumentMaurizio De Santis2013-02-071-0/+3
| | |
| * | Add documentation for ActiveRecord.select when use strings as argumentsMaurizio De Santis2013-02-071-0/+10
| | |
| * | Add ActiveRecord.count documentation when used on group relationsMaurizio De Santis2013-02-071-0/+6
| | |