aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | ActiveJob Integration TestsCristian Bica2014-09-1122-5/+627
| | | | | |
* | | | | | Speed up schema dumper testsJeremy Kemper2014-09-101-11/+9
| | | | | | | | | | | | | | | | | | | | | | | | Dump the standard schema once instead of redoing it per test
* | | | | | Include default column limits in schema.rbJeremy Kemper2014-09-103-8/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows :limit defaults to be changed without pulling the rug out from under old migrations that omitted :limit because it matched the default at the time.
* | | | | | MySQL: schema.rb now includes TEXT and BLOB column limits.Jeremy Kemper2014-09-103-8/+15
| | | | | |
* | | | | | MySQL: correct LONGTEXT and LONGBLOB limits from 2GB to their true 4GBJeremy Kemper2014-09-103-4/+8
|/ / / / /
* | | | | Merge pull request #16379 from JackDanger/update-preloader-documentationRafael Mendonça França2014-09-101-23/+32
|\ \ \ \ \ | | | | | | | | | | | | Updating Associations::Preloader docs
| * | | | | Updating Associations::Preloader docsJack Danger Canty2014-08-171-23/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Much of the previous documentation introduced features new in 2011. This commit refreshes it to provide clearer code examples and spends more time describing the normal case (preloaded associations) and less time describing the fallback. [ci skip]
* | | | | | Merge pull request #16867 from yahonda/use_drop_table_to_drop_sequencesRafael Mendonça França2014-09-103-5/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Replace drop sql statement to drop_table method
| * | | | | | Replace drop sql statement to drop_table methodYasuo Honda2014-09-113-5/+5
|/ / / / / / | | | | | | | | | | | | | | | | | | to drop sequences at the same time each tables dropped for Oracle
* | | | | | i18n guide: clarify interpolation rules in _html keys [ci skip]Xavier Noria2014-09-101-0/+16
| | | | | |
* | | | | | Merge pull request #16861 from suginoy/fix_typo_people_postRafael Mendonça França2014-09-101-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci skip]Correct variables in the sample code
| * | | | | | [ci skip]Correct variables in the sample codesuginoy2014-09-101-2/+2
| | | | | | |
* | | | | | | Tell the user which file is missing when config/database.yml was not foundAkira Matsuda2014-09-102-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since cc03675d30b58e28f585720dad14e947a57ff5b the error message became like "Could not load database configuration. No such file -" which doesn't really tell what's actually missing.
* | | | | | | Merge pull request #16858 from y-yagi/fix_web_console_docsGuillermo Iguaran2014-09-101-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | [ci skip] application.rb -> development.rb in web-console docs
| * | | | | | [ci skip] application.rb -> development.rb in web-console docsyuuji.yaginuma2014-09-101-1/+1
|/ / / / / /
* | | | | | Merge pull request #16855 from slavingia/patch-1Jeremy Kemper2014-09-091-1/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Removed any default margin from mail preview body.
| * | | | | | Removed any default margin from body.Sahil Lavingia2014-09-091-1/+5
|/ / / / / /
* | | | | | Oracle does not support `IF EXISTS` for `DROP VIEW`.Yves Senn2014-09-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | /cc @yahonda
* | | | | | models backed by views don't assume "id" columns are the primary key.Yves Senn2014-09-091-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #10247. The same goes for tables with an "id" column but without primary key constraint. Reading from the view works without configuration. If you have an updateable view you need to use `self.primary_key = ''`.
* | | | | | add test-cases for primary-key-less-views. Closes #16555.Yves Senn2014-09-091-0/+43
| | | | | |
* | | | | | Merge pull request #14056 from girishso/14041Yves Senn2014-09-094-4/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SQLite3Adapter now checks for views in table_exists? fixes: 14041 Conflicts: activerecord/CHANGELOG.md
| * | | | | | SQLite3Adapter now checks for views in table_exists? fixes: 14041Girish S2014-02-143-1/+17
| | | | | | |
* | | | | | | introduce `connection.supports_views?` and basic view tests.Yves Senn2014-09-096-4/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `AbstractAdapter#supports_views?` defaults to `false` so we have to turn it on in adapter subclasses. Currently the flag only controls test execution. /cc @yahonda
* | | | | | | Merge pull request #16849 from y-yagi/patch-1Vijay Dev2014-09-091-1/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove extra newline from ActiveJob test template
| * | | | | | | Remove extra newline from ActiveJob test templateyuuji.yaginuma2014-09-091-1/+0
| | | | | | | |
* | | | | | | | Merge pull request #16850 from aditya-kapoor/correct-lineVijay Dev2014-09-091-3/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | [ci skip] change the line orientation on asset precompile page
| * | | | | | | | [ci skip] change the line orientation on asset precompile pageAditya Kapoor2014-09-091-3/+1
| |/ / / / / / /
* / / / / / / / Allow included modules to override association methods.Yves Senn2014-09-095-2/+23
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #16684. This is achieved by always generating `GeneratedAssociationMethods` when `ActiveRecord::Base` is subclassed. When some of the included modules of `ActiveRecord::Base` were reordered this behavior was broken as `Core#initialize_generated_modules` was no longer called. Meaning that the module was generated on first access.
* | | | | | | :scissors: "now" [ci skip]Godfrey Chan2014-09-081-1/+1
| | | | | | | | | | | | | | | | | | | | | minor fix for #16846
* | | | | | | Merge pull request #16846 from ↵Godfrey Chan2014-09-081-1/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | prathamesh-sonpatki/document-error-handling-in-transactions-callbacks Update upgrading guide about error handling in transactional callbacks [ci skip]
| * | | | | | | Update upgrading guide about error handling in transactional callbacksPrathamesh Sonpatki2014-09-091-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Part of 16576 - [ci skip]
* | | | | | | | Merge pull request #16848 from prathamesh-sonpatki/kill-extra-has-beenGuillermo Iguaran2014-09-091-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove extra 'has been' from deprecation warning about asserting selectors
| * | | | | | | | Remove extra 'has been' from deprecation warning about asserting selectorsPrathamesh Sonpatki2014-09-091-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | Merge pull request #16847 from prathamesh-sonpatki/kill-extra-has-beenGuillermo Iguaran2014-09-091-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove extra 'has been' from the deprecation message
| * | | | | | | | Remove extra 'has been' from the deprecation messagePrathamesh Sonpatki2014-09-091-1/+1
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | - [ci skip]
* | | | | | | | Merge pull request #16845 from todd/upgrade_guide_console_docsGuillermo Iguaran2014-09-081-2/+6
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Add docs for web-console to 4.2 Upgrade Guide
| * | | | | | | Add docs for web-console to 4.2 Upgrade Guide [ci skip]Todd Bealmear2014-09-081-2/+6
|/ / / / / / /
* | | | | | | Do not memoize document_root_element in view testsRafael Mendonça França2014-09-082-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Memoizing will not make possible to assert the output of the view if it is changed after the first assert_select call Related with plataformatec/simple_form#1130 and rails/rails-dom-testing#15
* | | | | | | Merge pull request #16827 from kuldeepaggarwal/f-disable-migration-loggingYves Senn2014-09-081-1/+4
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | disable migrations logging while running test cases for AR schema tests
| * | | | | | disable migrations logging while running test cases for AR schema testsKuldeep Aggarwal2014-09-061-0/+2
| | | | | | |
* | | | | | | Merge pull request #16825 from cristianbica/fix-ar-nested-arraysMatthew Draper2014-09-084-6/+44
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix query with nested array in Active Record
| * | | | | | | Fix query with nested array in Active RecordCristian Bica2014-09-064-6/+44
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `User.where(id: [[1,2],3])` was equal to `User.where(id:[1, 2, 3])` in Rails 4.1.x but because of some refactoring in Arel this stopped working in 4.2.0. This fixes it in Rails. [Dan Olson & Cristian Bica]
* | | | | | | Merge pull request #16834 from tchandy/removing_unused_fake_modelsGuillermo Iguaran2014-09-071-45/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Removing unused fake models
| * | | | | | | Removing unused fake modelsThiago Pradi2014-09-071-45/+0
| | | | | | | |
* | | | | | | | Schema dumper: all connection adapters implement #primary_key, so rely on it ↵Jeremy Kemper2014-09-071-10/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | exclusively
* | | | | | | Merge pull request #16830 from teeparham/schema_dumperSantiago Pastorino2014-09-071-27/+28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Cleanup SchemaDumper
| * | | | | | | Fix warnings for undefined local variableTee Parham2014-09-071-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add private method primary_key_for, which more clearly shows that the expected return value is nil when a primary key is not found.
| * | | | | | | Prefer "if any?" to "unless empty?"Tee Parham2014-09-071-9/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Consistent whitespace * Remove unnecessary parentheses
| * | | | | | | Convert string concatenations to substitutionsTee Parham2014-09-071-15/+15
| |/ / / / / /
* | | | | | | Merge pull request #16832 from yuki24/use-match-via-all-in-custom-exceptions-appZachary Scott2014-09-061-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | [guides] Use `match ..., via: :all` in the custom exceptions app [ci skip]