なるようになるブログ

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

rails commit log流し読み(2018/09/03)

2018/09/03分のコミットです。

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

railties/CHANGELOG.md


Merge pull request #33773 from mcnelson/null_cache_store_for_test_env

railties/lib/rails/generators/rails/app/templates/config/environments/test.rb.ttの修正です。

test envのdefaultのcache storeにnull_storeを指定するよう修正しています。 Rails guideでtest envでnull storeを使用する事を推奨している箇所があり、ならそもそもdefaultの値もそうなっていて良いのでは、ということで修正されました。


Update changelog with null_store default

railties/CHANGELOG.mdの修正です。

先のcache storeのdefautの変更についてCHANGELOGにエントリーを追加しています。


Fix regression setting children record in parent before_save callback.

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

(09/03 07:55 修正)

親associationのbefore_save blockで子associationを設定、かつ、save処理で子associationでエラーがあった場合に、そのエラーの情報が正しく取得出来ないバグがあったのを修正しています。

親associationのbefore_save(=親associationのvalidationの後)で追加された子associationが、5.2.0までは子のsaveが成功してたのが5.2.1ではassociation.insert_recordで親のidがfillされる前に子のvalidationをするせいで子のsaveに失敗するというリグレッション、ということでした。 kamipoさんにご指摘頂きました。ありがとうございます。