aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Fix typo for api_app_generator_testYoshiyuki Hirano2017-08-151-1/+1
| | | | | |
* | | | | | Formatting CHANGELOGs [ci skip]Ryuta Kamizono2017-08-152-4/+4
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Add missing backticks Add missing * Add missing .
* | | | | Merge pull request #30259 from yhirano55/fix_format_of_generator_docsRyuta Kamizono2017-08-151-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | | Fix format of generator docs [ci skip]
| * | | | Fix format of generator docs [ci skip]Yoshiyuki Hirano2017-08-151-1/+1
|/ / / /
* | | | Merge pull request #30169 from awortham/awortham/awortham/fix-sql-distinct-bugRafael Mendonça França2017-08-143-0/+35
|\ \ \ \ | | | | | | | | | | | | | | | Ensure sum honors distinct on has_many through
| * | | | Ensure sum honors distinct on has_many throughAaron Wortham2017-08-143-0/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using a has_many through relation and then summing an attribute the distinct was not being used. This will ensure that when summing an attribute, the number is only used once when distinct has been used.
* | | | | Merge pull request #30215 from untidy-hair/clarify-abstract-or-not-in-testsRafael França2017-08-143-8/+18
|\ \ \ \ \ | | | | | | | | | | | | Clarify base_class tests on abstract STI vs concrete STI
| * | | | | Clarify base_class tests on abstract STI vs concrete STIYukio Mizuta2017-08-123-8/+18
| | | | | |
* | | | | | Merge pull request #30073 from yalab/fixture_binary_helperRafael França2017-08-144-1/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | Add `binary` helper method to fixtures.
| * | | | | | Add `binary` helper method to fixtures.yalab2017-08-124-1/+14
| | | | | | |
* | | | | | | Merge pull request #30219 from yhirano55/fix_app_generator_api_optionsRafael França2017-08-147-29/+48
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Refactor `--api` option of the app generator
| * | | | | | | Refactor `--api` option of the app generatorYoshiyuki Hirano2017-08-137-29/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It can't use JavaScript in api mode. Action cable should be disabled by default.
* | | | | | | | Merge pull request #30246 from yhirano55/fix_guide_command_lineRafael França2017-08-141-0/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix guide about command_line [ci skip]
| * | | | | | | | Fix guide about command_line [ci skip]Yoshiyuki Hirano2017-08-151-0/+3
| | |_|_|_|/ / / | |/| | | | | |
* | | | | | | | Merge pull request #30208 from kamipo/extract_primary_key_to_abstract_reflectionRafael França2017-08-141-8/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Extract `primary_key` to `AbstractReflection`
| * | | | | | | | Extract `primary_key` to `AbstractReflection`Ryuta Kamizono2017-08-121-8/+4
| | |_|_|/ / / / | |/| | | | | |
* | | | | | | | Merge pull request #27609 from kamipo/fix_association_primary_keyRafael França2017-08-144-21/+25
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix `reflection.association_primary_key` for `has_many` association
| * | | | | | | | Fix `reflection.association_primary_key` for `has_many` associationsRyuta Kamizono2017-08-134-21/+25
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is incorrect to treat `options[:primary_key]` as `association_primary_key` if `has_many` associations because the `:primary_key` means the column on the owner record, not on the association record. It will break `ids_reader` and `ids_writer`. ```ruby people(:david).essay_ids # => ActiveRecord::StatementInvalid: Mysql2::Error: Unknown column 'essays.first_name' in 'field list': SELECT `essays`.first_name FROM `essays` WHERE `essays`.`writer_id` = 'David' ``` Fixes #14439.
* | | | | | | | Merge pull request #30229 from kamipo/allow_serialize_with_custom_coderRafael França2017-08-143-9/+54
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Allow `serialize` with a custom coder on `json` and `array` columns
| * | | | | | | | Allow `serialize` with a custom coder on `json` and `array` columnsRyuta Kamizono2017-08-133-9/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We already have a test case for `serialize` with a custom coder in `PostgresqlHstoreTest`. https://github.com/rails/rails/blob/v5.1.3/activerecord/test/cases/adapters/postgresql/hstore_test.rb#L316-L335
* | | | | | | | | Merge pull request #30214 from kirs/uniqueness-scope-validatorRafael França2017-08-142-0/+11
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Check :scope input in Uniqueness validator
| * | | | | | | | | Check :scope input in Uniqueness validatorKir Shatrov2017-08-132-0/+11
| | |_|/ / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #30236 from padi/find-each-example-updateRafael França2017-08-141-1/+6
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | Completes ActiveRecord::Batches.find_each example [ci skip]
| * | | | | | | | Completes ActiveRecord::Batches.find_each example [ci skip]Marc Rendl Ignacio2017-08-131-1/+6
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous paragraph mentions that you can hand off the same processing queue to multiple workers. This completes the following example below it.
* | | | | | | | Merge pull request #30255 from lourenci/change-assets-path-of-pipelineVipul A M2017-08-151-2/+2
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | Fix asset pipeline guide [ci skip]
| * | | | | | | Fix asset pipeline guide [skip ci]Leandro Lourenci2017-08-141-2/+2
| | | | | | | |
* | | | | | | | Move test to its proper placeGeorge Claghorn2017-08-141-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ActiveStorage::Filename is a model, so its test case belongs in test/models.
* | | | | | | | Merge pull request #30231 from kamipo/remove_unused_source_type_infoRafael França2017-08-141-10/+7
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Remove unused `source_type_info` in `RuntimeReflection`
| * | | | | | | | Remove unused `source_type_info` in `RuntimeReflection`Ryuta Kamizono2017-08-131-10/+7
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `source_type_info` is only used for `constraints` in `PolymorphicReflection`.
* | | | | | | | Merge pull request #30244 from yhirano55/refactor_max_age_secondsRafael França2017-08-142-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Refactor max-age seconds
| * | | | | | | | Refactor max-age secondsYoshiyuki Hirano2017-08-142-2/+2
| | | | | | | | |
* | | | | | | | | Merge pull request #30233 from kamipo/remove_duplicated_join_id_forRafael França2017-08-141-10/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Remove duplicated `join_id_for`
| * | | | | | | | | Remove duplicated `join_id_for`Ryuta Kamizono2017-08-131-10/+2
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | The primary key on the owner record is abstracted as `join_foreign_key`.
* | | | | | | | | Merge pull request #30235 from bogdanvlviv/omit-additional-removing-cable_ymlRafael França2017-08-141-1/+0
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Omit additional removing `config/cable.yml`
| * | | | | | | | | Omit additional removing `config/cable.yml` by the app generatorbogdanvlviv2017-08-131-1/+0
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | `config/cable.yml` is never created if `--skip-action-cable` is given.
* | | | | | | | | Merge pull request #30239 from bogdanvlviv/improve-plugin_generator_codeRafael França2017-08-141-2/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Improve plugin generator codebase
| * | | | | | | | | Remove `:force` from PASSTHROUGH_OPTIONS for plugin generatorbogdanvlviv2017-08-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `:force` is setted inside the method `generate_test_dummy`
| * | | | | | | | | Register `--api` option for plugin generator by PASSTHROUGH_OPTIONS constantbogdanvlviv2017-08-131-2/+1
| | |/ / / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #30252 from rails/activestorage-service-ciRafael França2017-08-142-3/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Run Active Storage service tests in CI
| * | | | | | | | | Run Active Storage service tests in CIGeorge Claghorn2017-08-142-3/+1
| | | | | | | | | |
* | | | | | | | | | Merge pull request #30241 from yhirano55/optimize_routes_indentationRafael Mendonça França2017-08-145-34/+39
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Optimize routes indentation
| * | | | | | | | | | Optimize routes indentationYoshiyuki Hirano2017-08-145-33/+38
| | |_|/ / / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #29891 from pat/frozen-string-literals-railtiesRafael França2017-08-14311-18/+638
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Railties updates for frozen string literals.
| * | | | | | | | | | Allow for frozen string pragma in generated files.Pat Allan2017-08-141-2/+2
| | | | | | | | | | |
| * | | | | | | | | | Add frozen string pragma to expected routes files.Pat Allan2017-08-141-0/+2
| | | | | | | | | | |
| * | | | | | | | | | Adding frozen_string_literal pragma to Railties.Pat Allan2017-08-14311-1/+618
| | | | | | | | | | |
| * | | | | | | | | | Railties updates for frozen string literals.Pat Allan2017-08-1414-15/+16
| | |_|_|/ / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #30206 from yukihirop/improve_activerecord_rdocKasper Timm Hansen2017-08-141-0/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Add document that you rake test single file by using DB(ex. postgresql)
| * | | | | | | | | | Add document that you rake test single file by using DB(ex. postgresql) [ci ↵yukihirop2017-08-121-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | skip]
* | | | | | | | | | | Merge pull request #29994 from y-yagi/fix_test_runner_helpKasper Timm Hansen2017-08-143-22/+18
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | Show minitest options in test runner help
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-22 16:36:43 +0000