rails commit log流し読み(2017/03/22) - なるようになるブログ

2017/03/22分のコミットです。

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

Merge pull request #28450 from adamgamble/bug_fix/fix_typo_in_actionview

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

サニタイズしていないparameterを使用してボタンを生成しようとした場合のエラーメッセージで、buttonをbuttomにタイポしていたのを修正しています。

Revert “Merge pull request #27775 from domcleal/27774-format-reset”

ActionController::TestCase::Behavior#processメソッドで@requestのPATH_INFOを削除するよう修正した、Delete PATH_INFO after each controller test requestをrevetしています。

この対応により、リグレッション(request実行後にrequest pathを確認する事が出来なくなってしまった)が発生した為、revertしたとの事です。

Wrap store accessors in parameters

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

store_accessorを使用している、かつ、request parameterにstore_accessorのkeyに一致する値が含まれていた場合に、paramsにその値が正しく設定されないバグがあったのを修正しています。

class User < ApplicationRecord
  store_accessor :settings, :color, :size
end

post :parse, params: { "username" => "sikachu", "color" => "blue", "size" => "large" }

# before

params  # => username" => "sikachu", "color" => "blue", "size" => "large", "user" => { "username" => "sikachu" }

# after
params  # => username" => "sikachu", "color" => "blue", "size" => "large", "user" => { "username" => "sikachu", "color" => "blue", "size" => "large" }