なるようになるブログ

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

rails commit log流し読み(2018/07/13)

2018/07/13分のコミットです。

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


e4e1b62 broke to_param handling:

actionpack/lib/action_controller/test_case.rbactivesupport/lib/active_support/core_ext/object/to_query.rbの修正です。

Merge pull request #33093 from rails/use-rack-to-dump-queryで、ActionController::TestRequestクラスでqueryを生成するのにHash#to_queryメソッドを使用していたのをRack::Utils.build_nested_queryを使用するよう修正したのですが、これを再度to_queryメソッドを使用するよう修正しています。

Rack::Utils.build_nested_queryを使用する事により、paramsのvalueにActive Recordのオブジェクトを指定した場合の結果が変わってしまった(post :foo, params: { user: User.first }を指定した場合、元々は{ "user" => "1" }になっていたのが{ "user": "#<User: ...>" }になってしまった)為。

代わりに、Hash#to_queryでArray([])を含まない場合にのみソートを行うようにして対応しています。


Merge pull request #33340 from ttanimichi/patch-1

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

SanitizeHelper.sanitizer_vendorメソッドのdocに、互換性のために残しておりRails 5.1で削除する旨コメントがされていたのですが、現状まだメソッドが残っているため、コメント内の削除予定Railsのバージョンを5.1から6に更新しています。


Fix analyzing new blobs from uploaded files on attach

activestorage/lib/active_storage/attached/model.rbの修正です。

upload fileをattachした際に、blobのanalyze処理が正しく行われないバグがあったのを修正しています。