読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

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を使用してボタンを生成しようとした場合のエラーメッセージで、buttonbuttomにタイポしていたのを修正しています。


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

ActionController::TestCase::Behavior#processメソッドで@requestPATH_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" }

Fix test that was asserting the wrong thing

actionpack/test/controller/parameters/accessors_test.rbの修正です。

ActionController::Parametersのaccessorsについてのテストで、assertionで参照する変数名が誤っている箇所があったのを修正しています。


Fix test assertion

actionpack/test/controller/test_case_test.rbの修正です。

request実行後にpathが取得出来る事を確認するテストで、assertionに指定する値が間違えている箇所があったのを修正しています。


Start Rails 5.2 development

コンポーネントのバージョンを5.2.0 alphaに更新しています。これによりmasterブランチはもう5.2.0。


Update bug report templates

bug report templatesの修正です。使用するgemのバージョンを5.1.0.rc1に更新しています。


Merge pull request #28394 from shime/docs-action-dispatch

Action Packのdocの修正です。

ActionDispatch module配下のメソッド、クラス等の各doc内のタイポ、グラマー等をまとめて修正しています。


[PostgreSQL]: Replace deprecated PG constants.

Active Recordの修正です。

deprecatedになっていたpg gemの定数(PGconn, PGresult, PGError)を使ってたいたのを、使用しないよう修正しています。

参考:ruby-pg/History.rdoc at master · ged/ruby-pg


Don’t comment out config.file_watcher during Rails upgrade

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

app:update task実行時にconfig/environments/development.rb内のconfig.file_watcherの行を強制的にコメントアウトするようになっていたのを、コメントアウトしないよう修正しています。

Rails 4.2 -> 5.0にアップデート時に必要だった処理(listenがGemfileに無いのにEventedFileUpdateCheckerを使おうとするとエラーになってしまう為)だったのですが、5.0 -> 5.1へのアップデート時には不要な為削除しています。