なるようになるブログ

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

rails commit log流し読み(2017/06/24)

2017/06/24分のコミットです。

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


Removed reference to unexisting methods:

actionpack/lib/action_dispatch/testing/integration.rbの修正です。

ActionDispatch::Integration::Runner moduleで既に使用される事の無いxml_http_requestxhrget_via_redirectpost_via_redirectメソッドの定義処理があったのを削除しています。


Don’t wrap parameters if key already exists

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

ActionController::ParamsWrapperで、paramに既にkeyが存在している、かつ、値が nil / falseの場合に再度wrap処理を行ってしまうバグがあったのを修正しています。

{ "user" => nil } という値があった場合に{ "user" => { "user" => nil } }のようにwrapされてしまう、というバグでした(本来は{ "user" => nil }がそのまま返ってくるべき)。parameterの存在チェックに#[]で返ってきた値をそのまま使用していた為問題が発生していました。


Guides: Clarify partial local variable naming.

rails guideのLayouts and Rendering in Railsの修正です。

Passing Local Variablesの項、partial templateのローカル変数について説明している箇所の説明を修正しています。