なるようになるブログ

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

rails commit log流し読み(2017/11/08)

2017/11/08分のコミットです。

CHANGELOGへの追加はありませんでした。


Do not run active_storage:install when bundle install is skipped

railties/lib/rails/generators/app_base.rbの修正です。

bundle installがスキップされた場合、active_storage:installタスクを実行しないよう修正しています。

active_storage:installrailsコマンドを使用して実行されるのですが、bundle installが実行されていない場合railsコマンドは失敗してしまう(可能性がある)為。


Remove unnecessary migration deletion

railtiesのテストの修正です。

テストの各箇所でdb/migrateを削除していたのを、削除しないよう修正しています。

active_storage:installタスクでインストールされたmigrationファイルを削除する為に上記処理を行っていたのですが、先のコミットの影響でテスト中ではactive_storage:installタスクが実行されなくなり、migrationファイルの削除処理が不要になった為。


Fix output of select_tag with include_blank: true [ci skip]

actionview/lib/action_view/helpers/form_tag_helper.rbのdocの修正です。

select_tagメソッドのdoc内、include_blank: trueを指定した場合のexampleにlabel属性を追加しています。

Add empty label to empty options taglabel属性も強制的に出力するよう挙動が変更になった為。


Use released redis-namespace instead of master version

Gemfileの修正です。

redis-namespaceを使用するのにGitHubのmaterブランチを直接指定していたのを、リリース済みのgemを使用するよう修正しています。


Don't expose internal methods in Preloader::ThroughAssociation

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

内部でだけで使用するPreloader::ThroughAssociation#through_reflection#source_reflectionメソッドの可視性をprivateに変更しています。


Don't expose accessors which are internal used only

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

内部でだけで使用する accessorsの可視性をprotectedに変更しています。