aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Update rails-ujs readmeMike Fiedler2018-01-291-1/+1
| | | Link to W3C reference was broken, this uses the latest URL, along with HTTPS.
* Merge pull request #31814 from fatkodima/index-nulls-orderRafael França2018-01-293-2/+40
|\ | | | | Dump correctly index nulls order for PostgreSQL
| * Dump correctly index nulls order for postgresqlfatkodima2018-01-283-2/+40
| |
* | Merge pull request #31821 from composerinteralia/extra-to_sGeorge Claghorn2018-01-298-9/+9
|\ \ | | | | | | Avoid extra calls to to_s
| * | Avoid extra calls to to_sDaniel Colson2018-01-298-9/+9
| | | | | | | | | | | | | | | | | | | | | With #31615 `type_for_attribute` accepts either a symbol as well as a string. `has_attribute?` and `attribute_alias` also accept either. Since these methods call `to_s` on the argument, we no longer need to do that at the call site.
* | | Merge pull request #31825 from mistydemeo/fix_post_documentationEileen M. Uchitelle2018-01-291-1/+1
|\ \ \ | | | | | | | | ActionController::TestCase: fix #post documentation
| * | | ActionController::TestCase: fix #post documentation [ci skip]Misty De Meo2018-01-291-1/+1
|/ / / | | | | | | | | | Fixes #31823.
* / / Deprecate `expand_hash_conditions_for_aggregates`Ryuta Kamizono2018-01-293-0/+13
|/ / | | | | | | | | This can not expand an array of aggregated objects correctly, and is no longer used internally since 159b21b5.
* | Suppress expected exceptions by `report_on_exception` = `false`yuuji.yaginuma2018-01-291-0/+4
| | | | | | | | | | This suppresses `Timeout::Error` exceptions. Ref: https://travis-ci.org/rails/rails/jobs/334622442#L1089-L1110
* | Fix the example of 'finding records by a value object' (#31818)Bogdan2018-01-291-2/+2
| | | | | | | | | | | | | | | | | | This example was added in abdf546ad6d02ecb95766e73cd3c645a48c954de but was inconsistent with `composed_of :balance` definition in the 'Customer'. [Ryuta Kamizono & bogdanvlviv] [ci skip]
* | Extract `discard_remaining_results` for mysql2 adapterRyuta Kamizono2018-01-292-4/+6
| |
* | Revert "Merge pull request #23256 from pauloancheta/master"Ryuta Kamizono2018-01-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 755f7b58953c25790eea0574ff8928033ded2d88, reversing changes made to 35db4603095e8ccc0a92d925a42d4b3d9462580f. Originally `mapping: %w(balance amount)` is correct mapping example. Closes #31346. [Ryuta Kamizono & bogdanvlviv] [ci skip]
* | Remove unused/missing `to_money` converter in the test/docRyuta Kamizono2018-01-292-3/+2
| |
* | Merge pull request #31615 from cben/type_for_attribute-symbolRyuta Kamizono2018-01-292-1/+16
|\ \ | | | | | | Allow type_for_attribute(:symbol)
| * | Allow type_for_attribute(:symbol)Beni Cherniavsky-Paskin2018-01-012-1/+16
| | | | | | | | | | | | Closes #31611.
* | | Merge pull request #31810 from shuheiktgw/add_tests_for_blobGeorge Claghorn2018-01-281-0/+12
|\ \ \ | | | | | | | | Added tests for ActiveStorage::Blob#image? and ActiveStorage::Blob#video?
| * | | Add a test for ActiveStorage::Blob#image? and ActiveStorage::Blob#video?Shuhei Kitagawa2018-01-291-0/+12
| | | |
* | | | Fix expanding an array of `composed_of` objects which have multiple mappingsRyuta Kamizono2018-01-295-6/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Follow up of #31724. If `composed_of` objects have multiple mappings, array predicate handler can not correctly handle the expanded condition. We need to handle it like polymorphic association objects.
* | | | Merge pull request #31817 from composerinteralia/mediocre-jokeRichard Schneeman2018-01-281-1/+1
|\ \ \ \ | | | | | | | | | | Remove joke in security guide [ci skip]
| * | | | Remove joke in security guide [ci skip]Daniel Colson2018-01-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I think this is a joke, although not a great one. It's mildly unprofessional, so I think we should get rid of it.
* | | | | Merge pull request #31724 from orekyuu/fix-expand-composed-object-arrayRyuta Kamizono2018-01-293-3/+37
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix not expanded problem when passing an Array object as argument to the where method using composed_of column.
| * | | | | Allow expanding an array of `composed_of` objectsRyuta Kamizono2018-01-292-9/+6
| | | | | |
| * | | | | Fix not expanded problem when passing an Array object as argument to the ↵orekyuu2018-01-263-3/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | where method using composed_of column. Fixes #31723 ``` david_balance = customers(:david).balance Customer.where(balance: [david_balance]).to_sql # Before: WHERE `customers`.`balance` = NULL # After : WHERE `customers`.`balance` = 50 ```
* | | | | | Merge pull request #31811 from ↵Ryuta Kamizono2018-01-291-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | yahonda/supports_foreign_tables_in_abstract_adapter Define `supports_foreign_tables?` in AbstractAdapter
| * | | | | | Define `supports_foreign_tables?` in AbstractAdapterYasuo Honda2018-01-281-0/+5
| | |_|_|_|/ | |/| | | |
* | | | | | Merge pull request #31816 from composerinteralia/jquery-referencesRyuta Kamizono2018-01-291-11/+6
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Remove out-of-date references to jQuery [ci skip]
| * | | | | Remove out-of-date references to jQueryDaniel Colson2018-01-281-11/+6
| | |/ / / | |/| | | | | | | | | | | | | Fixes #31703
* | | | | Update `action_dispatch.default_headers` default value [ci skip]yuuji.yaginuma2018-01-282-2/+8
| |_|/ / |/| | | | | | | | | | | This was changed with 5d7b70f and 428939b.
* | | | Update `google-cloud-storage` gem version [ci skip]yuuji.yaginuma2018-01-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Because we use `credentials` in an example of configuration files, but `credentials` can not be used unless it is 1.8.0 or later. Ref: 9d65ac3
* | | | Eliminate ActiveStorage::Service::MirrorServiceTest#uploadShuhei Kitagawa2018-01-271-16/+16
| | | |
* | | | Merge pull request #31807 from composerinteralia/minitest-versionsGuillermo Iguaran2018-01-262-2/+0
|\ \ \ \ | | | | | | | | | | Relax minitest versions
| * | | | Relax minitest versionDaniel Colson2018-01-262-2/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | We locked Minitest to 5.11.1 in #31799 because 5.11.2 included a breaking change. The change was fixed in 5.11.3, so we no longer need to lock in the version.
* / / / Unlink internal tempfiles after useGeorge Claghorn2018-01-262-2/+22
|/ / /
* | | Merge pull request #31803 from rmosolgo/rm-dependenciesRafael França2018-01-262-0/+56
|\ \ \ | | | | | | | | Fix infinite loop when unloading autoloaded modules
| * | | Remove duplicates after autoloading modulesRobert Mosolgo2018-01-261-0/+1
| | | |
| * | | Add failing test for infinite loop when unloading autoloaded modules when an ↵Robert Mosolgo2018-01-261-0/+55
| | | | | | | | | | | | | | | | error occured during the load.
* | | | Merge pull request #31800 from rlue/doc/association-basics-guideRafael França2018-01-261-17/+9
|\ \ \ \ | | | | | | | | | | Fix example migrations in Associations guide
| * | | | Fix example migrations in Associations guideRyan Lue2018-01-261-17/+9
| | | | |
* | | | | Merge pull request #31786 from composerinteralia/respond-to-testsRafael França2018-01-26240-2237/+2237
|\ \ \ \ \ | |/ / / / |/| | | | Use respond_to test helpers
| * | | | Remove extra whitespaceDaniel Colson2018-01-2515-95/+95
| | | | |
| * | | | Use assert_empty and assert_not_emptyDaniel Colson2018-01-2551-194/+194
| | | | |
| * | | | Use assert_predicate and assert_not_predicateDaniel Colson2018-01-25209-1988/+1988
| | | | |
| * | | | Change refute to assert_notDaniel Colson2018-01-2519-51/+51
| | | | |
| * | | | Use respond_to test helpersDaniel Colson2018-01-2543-158/+158
|/ / / /
* | | | Bring back ability to insert zero value on primary key for fixtures (#31795)Ryuta Kamizono2018-01-265-42/+22
| | | | | | | | | | | | | | | | | | | | | | | | Since #29504, mysql2 adapter lost ability to insert zero value on primary key due to enforce `NO_AUTO_VALUE_ON_ZERO` disabled. That is for using `DEFAULT` on auto increment column, but we can use `NULL` instead in that case.
* | | | Merge pull request #31799 from yahonda/use_minitest_5_11_1Ryuta Kamizono2018-01-262-0/+2
|\ \ \ \ | |_|/ / |/| | | Lock Minitest 5.11.1
| * | | Lock Minitest 5.11.1Yasuo Honda2018-01-262-0/+2
|/ / / | | | | | | | | | | | | To workaround `undefined method `error?' for` reported at https://travis-ci.org/rails/rails/jobs/333456146
* | | Merge pull request #31797 from eugeneius/mysql_index_orderRyuta Kamizono2018-01-261-1/+1
|\ \ \ | | | | | | | | Update note on MySQL index order support [ci skip]
| * | | Update note on MySQL index order support [ci skip]Eugene Kenny2018-01-261-1/+1
|/ / / | | | | | | | | | | | | MySQL supports descending indexes from 8.0.1 onwards: https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-1.html
* | | Merge pull request #31755 from timdiggins/adjust-autoloading-in-guidesRafael França2018-01-252-12/+31
|\ \ \ | | | | | | | | Clarify autoload_paths and eager_load in guides per current usage.
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-07 09:22:18 +0000