aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Remove `null_allowed` option from doc [ci skip]yuuji.yaginuma2017-06-101-2/+1
|/ / / / / | | | | | | | | | | | | | | | This option was added in b9fa354. But it does not seem to work.
* | | | | Split up the cache test suite so it's easier to understand and extend (#29404)Jeremy Daer2017-06-1020-1359/+1383
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Split up the caching tests as prep for adding a new cache store. Slices the mega test/caching_test.rb into behavior modules, concrete store tests, and cross-cutting store tests. Considering moving cache store behavior modules into lib/ so they may be used for acceptance testing by third parties.
* | | | | Merge pull request #29403 from yasslab/add-backquote-to-counter-cacheRichard Schneeman2017-06-091-1/+1
|\ \ \ \ \ | | | | | | | | | | | | [ci skip] Add backquote to :counter_cache option
| * | | | | [ci skip] Add backquote to :counter_cache optionYohei Yasukawa2017-06-101-1/+1
|/ / / / /
* | | | | Make i18n test match the description of the testGuillermo Iguaran2017-06-091-1/+1
| | | | |
* | | | | Merge pull request #29361 from metaskills/sqlserversetup-take2Guillermo Iguaran2017-06-091-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Ensure MSSQL password uses real ENV var.
| * | | | | Ensure MSSQL password uses real ENV var.Ken Collins2017-06-051-2/+2
| | | | | |
* | | | | | Merge pull request #29398 from ydakuka/patch-1Richard Schneeman2017-06-091-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci skip] Fix another link in active_record_postgresql.md
| * | | | | | [ci skip] Fix link in active_record_postgresql.mdYauheni Dakuka2017-06-091-1/+1
|/ / / / / /
* | | | | | Merge pull request #29397 from ydakuka/patch-1Vipul A M2017-06-091-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix link in active_record_postgresql.md [ci skip]
| * | | | | | Fix link in active_record_postgresql.mdYauheni Dakuka2017-06-091-1/+1
| | | | | | |
* | | | | | | Merge pull request #29396 from ydakuka/patch-2Vipul A M2017-06-091-1/+1
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | remove the extra comma in association_basics.md [ci skip]
| * | | | | | remove the extra comma in association_basics.mdYauheni Dakuka2017-06-091-1/+1
|/ / / / / /
* | | | | | Don't rely on the @view_renderer being defined.Kasper Timm Hansen2017-06-081-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | That won't be true for Action Pack and Action Mailer.
* | | | | | Merge pull request https://github.com/rails/rails/pull/28637 from ↵Kasper Timm Hansen2017-06-086-33/+42
| | | | | | | | | | | | | | | | | | | | | | | | st0012/fix-partial-cache-logging
* | | | | | Use a hash to record every partial's cache hit status instead of sharing a ↵Stan Lo2017-06-087-3/+51
| | | | | | | | | | | | | | | | | | | | | | | | boolean.
* | | | | | Merge pull request #29388 from y-yagi/bump_capybaraKasper Timm Hansen2017-06-081-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Bundle capybara 2.14.1 that includes fix for Ruby warnings
| * | | | | | Bundle capybara 2.14.1 that includes fix for Ruby warningsyuuji.yaginuma2017-06-081-2/+2
|/ / / / / / | | | | | | | | | | | | | | | | | | Ref: https://github.com/teamcapybara/capybara/pull/1868
* | | | | | Merge pull request #29386 from y-yagi/remove_unreachable_codeGuillermo Iguaran2017-06-071-8/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove unreachable code
| * | | | | | Remove unreachable codeyuuji.yaginuma2017-06-081-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Time.find_zone!` raise `ArgumentError` if invalid value is specified. https://github.com/rails/rails/blob/379a0b42daf0d8e14130db7fd886d05d8d88e3f2/activesupport/lib/active_support/core_ext/time/zones.rb#L97..L99 Therefore, the return value never becomes nil.
* | | | | | | Don't support namespace in form_with.Kasper Timm Hansen2017-06-071-18/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | form_with requires people to pass an id manually, so users can just prefix their namespace right there.
* | | | | | | Move slicing to initializer.Kasper Timm Hansen2017-06-072-2/+2
| |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Forgot all about https://github.com/rails/rails/pull/28844/files#r113780934 cc @rafaelfranca
* | | | | | Cache: write_multi (#29366)Jeremy Daer2017-06-063-6/+100
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails.cache.write_multi foo: 'bar', baz: 'qux' Plus faster `fetch_multi` with stores that implement `write_multi_entries`. Keys that aren't found may be written to the cache store in one shot instead of separate writes. The default implementation simply calls `write_entry` for each entry. Stores may override if they're capable of one-shot bulk writes, like Redis `MSET`.
* | | | | | Merge pull request #29377 from edwardmp/fix-upgrading-docsRafael França2017-06-061-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Update upgrading guide w.r.t. Parameters to use other example method …
| * | | | | Update upgrading guide w.r.t. Parameters to use other example method than ↵edwardmp2017-06-061-1/+1
|/ / / / / | | | | | | | | | | | | | | | slice as this has actually been implemented by Parameters
* | | | | Merge pull request #28868 from yahonda/pg10_oid_unknown_to_textMatthew Draper2017-06-071-8/+8
|\ \ \ \ \ | | | | | | | | | | | | PostgreSQL 10 converts unknown type to text type
| * | | | | PostgreSQL 10 converts unknown OID 705 to text 25Yasuo Honda2017-06-061-8/+8
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Rename test cases from `unknown` to `unrecognized` since unknown OID is one of possible unrecognized types by Rails - Use "select 'pg_catalog.pg_class'::regclass" whose OID is 2205, which will not be converted to recognized type in PostgreSQL 10. activerecord_unittest=# select oid, typname from pg_type where oid in (2205, 2277); oid | typname ------+---------- 2205 | regclass 2277 | anyarray (2 rows) Addresses #28868
* | | | | Remove invalid entry from the guides index pageRobin Dupret2017-06-061-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 1a5d9399 removed the "Profiling Rails Applications" guide but the YAML entry for this guide wasn't. [ci skip]
* | | | | Tiny documentation fixes [ci skip]Robin Dupret2017-06-062-6/+6
| | | | |
* | | | | Merge pull request #29362 from kamipo/remove_redundant_assert_nothing_raisedMatthew Draper2017-06-063-11/+0
|\ \ \ \ \ | | | | | | | | | | | | Remove redundant `assert_nothing_raised` before another assertions
| * | | | | Remove redundant `assert_nothing_raised` before another assertionsRyuta Kamizono2017-06-063-11/+0
| | |/ / / | |/| | | | | | | | | | | | | These `assert_nothing_raised` are covered by following assertions.
* | | | | Merge pull request #29370 from koic/fix_test_pluck_without_column_names_oracleMatthew Draper2017-06-061-2/+5
|\ \ \ \ \ | |_|/ / / |/| | | | Fix `test_pluck_without_column_names` when using Oracle
| * | | | Fix `test_pluck_without_column_names` when using OracleKoichi ITO2017-06-061-2/+5
| | | | |
* | | | | Let's test nokogiri 1.8 against edge Rails, and vice versaAkira Matsuda2017-06-061-8/+8
| | | | |
* | | | | Merge pull request #29364 from lostapathy/patch-2Ryuta Kamizono2017-06-061-1/+0
|\ \ \ \ \ | | | | | | | | | | | | Remove html tag making markdown misrender [ci skip]
| * | | | | Remove html tag making markdown misrender [ci skip]Joe Francis2017-06-051-1/+0
|/ / / / /
* / / / / Fix indentation + Add backticks [ci skip]Ryuta Kamizono2017-06-061-2/+4
|/ / / /
* / / / [ci skip] UNIQUE constraint affects not only INSERT but also UPDATERyuta Kamizono2017-06-061-1/+1
|/ / /
* | | Merge pull request #29356 from ↵Eileen M. Uchitelle2017-06-052-151/+132
|\ \ \ | | | | | | | | | | | | | | | | kamipo/testing_reserved_word_test_for_all_adapters Testing `ReservedWordTest` for all adapters
| * | | Testing `ReservedWordTest` for all adaptersRyuta Kamizono2017-06-052-151/+132
| | | | | | | | | | | | | | | | | | | | `ReservedWordTest` expects that any identifiers are quoted properly. It should be tested for all adapters.
* | | | Remove deprecated passing of string to :if/:unless (#29357)Joe Francis2017-06-061-1/+1
|/ / / | | | | | | | | | | | | Other examples were removed in 53ff5fc62f9d11b6f60d371df959137f4bf40728 [ci skip]
* | | Merge pull request #29347 from prathamesh-sonpatki/rm-assert-nothin-raisedEileen M. Uchitelle2017-06-041-3/+3
|\ \ \ | | | | | | | | Remove assert_nothing_raised, as test is already testing the required concerns
| * | | Remove assert_nothing_raised, as test is already testing the required concernsPrathamesh Sonpatki2017-06-031-3/+3
| | | |
* | | | Don't mark the schema loaded until it's really finishedMatthew Draper2017-06-051-3/+5
| | | | | | | | | | | | | | | | | | | | load_schema! is overridden by attribute modules, so we need to wait until it has returned.
* | | | Merge pull request #29294 from gsamokovarov/attributes-defaultKasper Timm Hansen2017-06-0439-150/+134
|\ \ \ \ | | | | | | | | | | Introduce mattr_accessor default option
| * | | | Use mattr_accessor default: option throughout the projectGenadi Samokovarov2017-06-0336-125/+69
| | | | |
| * | | | Implement mattr_acessor :default optionGenadi Samokovarov2017-06-033-25/+65
| | | | |
* | | | | Merge pull request #29351 from vipulnsward/unnecessary-assignmentMatthew Draper2017-06-041-3/+3
|\ \ \ \ \ | | | | | | | | | | | | Don't create extra assignment, just return
| * | | | | Don't create extra assignment, just returnVipul A M2017-06-041-3/+3
|/ / / / /
* | | | | Merge pull request #29348 from andrehjr/fix-typo-route-definition-ambiguousMatthew Draper2017-06-041-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Fix typo on error message when route definition is ambiguous.