developer.strict_loading!(mode: :n_plus_one_only)

developer.projects.to_a # Does not raise
developer.projects.first.client # Raises StrictLoadingViolationError

ActiveRecord: Optimize cache_key computation (#41741)

activerecord/lib/active_record/relation.rbの修正です。

cache keyの生成の際に不要なqueryが実行されないよう修正しています。

[ci skip] Fix Grammatical Errors and Eliminate Ambiguous Wordings in Multiple Database Documents (#41670)

rails guideのMultiple Databases with Active Recordの修正です。

各箇所のグラマー、フォーマットの修正を行っています。

Revert "Prevent double save of cyclic associations"

associationの定義が循環している場合にauto saveでrecordが二重に保存されてしまうバグがあったのを修正した、Prevent double save of cyclic associationsをRevertしています。

associationの外部キーを正しく設定出来ておらず、associationの保存に失敗してしまうケースがあった為。

2021-03-25

rails commit log流し読み(2021/03/24)

2021/03/24分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activerecord/CHANGELOG.md

Prevent double saves in autosave of cyclic associations

actionpack/CHANGELOG.md

Deprecate the ability to assign a single value to config.action_dispatch.trusted_proxies as RemoteIp middleware behaves inconsistently depending on whether this is configured with a single value or an enumerable.

Fix SQL result of Book.joins(reviews: :customer) query example

rails guideのActive Record Query Interfaceの修正です。

Joining Nested Associations (Single Level)の項にあるexampleコードの実行結果が実際の結果と異なっていたのを修正しています。

Prevent double save of cyclic associations

Active Recordの修正です。

associationの定義が循環している場合に、auto saveでrecordが二重に保存されてしまうバグがあったのを修正しています。

Exclude #saving? from API docs

activerecord/lib/active_record/autosave_association.rbのdocの修正です。

内部APIであるAutosaveAssociation#saving?メソッドがAPI docに表示されないよう修正しています。

protected :can_save?

activerecord/lib/active_record/autosave_association.rb、 activerecord/test/cases/autosave_association_test.rbの修正です。

can_save?メソッドの可視性をprotectedに変更、及び、テストから不要なグローバル変数の削除等を行っています。

Add _ prefix for the internal methods

activerecord/lib/active_record/autosave_association.rbの修正です。

内部用のメソッドに_ prefixを追加しています。同名のメソッドがユーザに定義されて処理が上書きされてしまうのを避ける為。

Merge pull request #41725 from okuramasafumi/add-more-examples-to-routes-shallow-doc

rails guideのRails Routing from the Outside Inの修正です。

Shallow Nestingの項に、サンプルのroutesで生成されるroutingの一覧を追加しています。

Document Destroy Async (#41680)

rails guideのActive Record Associationsの修正です。

dependentのオプションについて説明している箇所に、:destroy_asyncについての説明を追加しています。

Merge pull request #41745 from dylanahsmith/ar-class-inspection-filter

activerecord/lib/active_record/core.rbの修正です。

ActiveRecord::Base.filter_attributesにProcを指定している場合、Active Record objectのmarshal dump処理がエラーになってしまうバグがあったのを修正しています。

Merge pull request #40789 from csutter/deprecate-remote-ip-single-value

actionpack/lib/action_dispatch/middleware/remote_ip.rbの修正です。

config.action_dispatch.trusted_proxiesにsingle valueを指定するがdeprecateになりました。config.action_dispatch.trusted_proxiesにsingle valueを指定した場合とenumerableを指定した場合とで挙動が違う(デフォルトの値に追加する、とデフォルトの値を上書きする)のが混乱の元であり、single valueの方の挙動を維持する必要は無いだろう、との事でdeprecateになりました。

2021-03-24

rails commit log流し読み(2021/03/23)

2021/03/23分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activesupport/CHANGELOG.md

Tests parallelization is now disabled when running individual files to prevent the setup overhead.

Revert "Fix schema for members"

テスト用のデータの型が誤っているのを修正した、Fix schema for members をRevertしています。

型が違うのは意図的だった(foreign_keyがStringの場合もロード出来る事を確認するテストだった)為。

Fix the test case for #14855 to catch a future regression correctly

activerecord/test/cases/associations/belongs_to_associations_test.rbの修正です。

polymorphic associationでforeign_keyがStringの場合のテストが期待通りにテストされてなかったのを修正しています。

Fix typo in 6.1 release notes

rails guideのRuby on Rails 6.1 Release Notesの修正です。

I18n.default_localeをI18n.default_localにタイポしていたのを修正しています。

Batch queries that are the same but are using different extension

activerecord/lib/active_record/associations/preloader/association.rb、 activerecord/lib/active_record/relation.rbの修正です。

preload batche処理で、extending、 skip_query_cache、strict_loadingが違うが生成されるqueryが同じ場合、batch処理でqueryがマージされるよう修正しています。

Merge pull request #41080 from ankurp/standardrb-fixes

railtiesの修正です。

generatorが生成するコードでシングルクオートを使用していた箇所を、ダブルクォートを使用するよう修正しています。

Remove non-project specific entry from gitignore

.gitignoreの修正です。

先のコミットで誤って.gitignoreに.DS_Storeが追加されてしまっていたのを削除しています。

More consistent quotes in template files

railtiesの修正です。

generatorが生成するコードでシングルクオートを使用していた箇所が残っていたのを、ダブルクォートを使用するよう修正しています。

Really make OrderedHash private to the framework

activesupport/lib/active_support/ordered_hash.rbのdocの修正です。

OrderedHashに:nodoc:を指定して、API docに表示されないよう修正しています。

Fix test_from_trusted_xml_allows_symbol_and_yaml_types failure

activesupport/test/core_ext/hash_ext_test.rbの修正です。

不足していたactive_support/ordered_hashのrequireを追加しています。

Parsing type="yaml" node requires require "yaml"

activesupport/test/core_ext/hash_ext_test.rbの修正です。

不足していたのはactive_support/ordered_hashではなく、yamlだったようで、active_support/ordered_hashのrequireを削除し、代わりにyamlのrequireを追加しています。

to_yaml requires require "yaml"

activesupport/test/hash_with_indifferent_access_test.rbの修正です。

不足していたyamlのrequireを追加しています。

Fix WhereClause#extract_attributes to work it with a string where clause

activerecord/lib/active_record/relation/where_clause.rbの修正です。

where clauseにStringが指定された場合に、WhereClause#extract_attributesがエラーになってしまうバグがあったのを修正しています。

Merge pull request #41736 from lairtonmendes/6-1-stable

rails guideのAction Mailer Basicsの修正です。

Walkthrough to Generating a Mailerの項にあるgeneratorのexampleから、不要なsuffixの指定を削除しています。

Use triple-dot delegation in ForkTracker

activesupport/lib/active_support/fork_tracker.rbの修正です。

引数のdelegationに...を使用するよう修正しています。

Merge pull request #41713 from henrik/lookup

docの修正です。

lookup -> look upに修正しています。

Merge pull request #41684 from ricardotk002/disable-parallel-testing

Active Support、railtiesの修正です。

test runnerに個別のファイルを指定された場合(e.g. ./bin/rails test test/controllers/my_controller_test.rb)はparallelization処理を行わないよう修正しています。テスト対象が1つの場合、parallel test用のセットアップ処理の方が実行コストよりも高い(可能性がある)為。