なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2017/07/01)

2017/07/01分のコミットです。

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

railties/CHANGELOG.md


Add CHANGELOG for #29630 [ci skip]

actionpack/CHANGELOG.mdの修正です。

先日コミットされた、ActionController::Parameters#to_sHash#to_sdelegateする対応について、CHANGELOGにエントリーを追加しています。


Delete stale comment for AR::Associations::Builder::CollectionAssociation

activerecord/lib/active_record/associations/builder/collection_association.rbのdocの修正です。

ActiveRecord::Associations::Builder::CollectionAssociationのdocに、has_manyhas_many_and_belongs_to_many associationsのclassがCollectionAssociationを継承している旨コメントがあったのですが、has_and_belongs_to_manyCollectionAssociationをもう継承していない為、コメントを削除しています。


Fix format of ActionController::Parameters#to_s doc [ci skip]

actionpack/lib/action_controller/metal/strong_parameters.rbのdocの修正です。

ActionController::Parameters#to_sメソッドのフォーマットが崩れていたのを修正しています。


Merge pull request #29629 from koic/fix_next_version_of_rails

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

Deprecation#initializedeprecation_horizon引数のデフォルト値を5.3から6.0に変更しています。5.2の次は6.0になる為。


Merge pull request #29536 from nickrivadeneira/fix-yaml

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

Active RecordのobjectをYAML.dumpしたデータををYAML.loadで戻した際に、元のattributeの値がfalseだった場合に、nilになってしまうバグがあったのを修正しています。


Merge pull request #28808 from fschwahn/fix-polymorphic-automic-inverse

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

polymorphic associationを使用している場合に、associationのbuildメソッドでインスタンスの生成が正しく行えないバグがあったのを修正しています。


Merge pull request #29644 from wilson/unify-route-helper-visibility

actionpack/lib/action_dispatch/routing/route_set.rbの修正です。

directメソッドで登録されたnamed helperを専用の変数(@custom_helpers)に登録していたのを、他のnamed helperと同じ変数に値を登録するように修正しています。

専用の変数に登録してしまうと、route_defined?等のメソッドで値を正しくチェック・取得が出来ない為との事です。


Merge pull request #29634 from kamipo/dont_cache_queries_for_schema_statements

Active Recordの修正です。

schema statementsでquery cacheを使用しないよう修正しています。

schema statementsの実行の際にquery cacheが使用されてしまう事により不安定なテストがある為、それを回避する為に対応したようです。


Merge pull request #29631 from kamipo/should_be_clear_association_ids

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

collection associationに新しいレコードを追加した際に、association_idsメソッドの値が更新されないバグがあったのを修正しています。


Add source code and changelog link to railties.gemspec

railties/railties.gemspecの修正です。

railtiesのgemspecに、metadataとしてsource_code_urichangelog_uriを設定しています。

Merge pull request #29588 from greysteil/add-gemspec-linksの対応漏れ。


Merge pull request #29534 from y-yagi/clear_screenshots_in_tmp_clear_task

railties/lib/rails/tasks/tmp.rakeの修正です。

tmp:clear taskがtmp/screenshotsディレクトリ配下のファイル(system testが生成するスクリーショットのファイル)を削除するよう修正しています。


SystemTestCase undef some IntegrationTest methods because it’s confused to use.

actionpack/lib/action_dispatch/system_test_case.rbactionpack/lib/action_dispatch/system_testing/test_helpers/undef_methods.rbの修正です。

System Testでhttp verb method(getpost等)を使えないようにする為に、左記メソッドをundef_methodするよう修正しています。

System TestではCapybaraのAPIを使用する事が推奨されている為。