なるようになるブログ

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

rails commit log流し読み(2015/08/16)

2015/08/16分のコミットです。

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


Initialize symbols instead of mapping to_sym on the set of strings

actionpack/lib/abstract_controller/rendering.rbの修正です。

AbstractControllerクラスのDEFAULT_PROTECTED_INSTANCE_VARIABLES定数を定義する際、%wでStringのArray定義し、そのArrayに対してto_symメソッドを呼び出してSymbolに変換していたのを、%i記法を使用して、最初からSymbolとして定義するよう修正しています。


Refactor how assign_parameters sets generated_path & query_string_keys

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

assign_parametersメソッド内でまとめて行っていたcontrollerクラス名の取得、pathの生成等の処理を別メソッドに切り出しています。

controller testsのリファクタリングをやる予定があり、その一部との事です。


Fix test assign_parameter method signature

actionview/test/template/test_case_test.rbの修正です。

先のコミットでassign_parametersメソッドシグネチャを変更しており、その影響でコケてしまったテストがあったのを修正しています。


only process via once

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

viaオプションに関する処理をコンストラクタapplicationメソッドそれぞれで行っていたのを、一度だけ行うよう修正しています。


use the mapper to build the routing table

actionpack/test/journey/router_test.rbの修正です。

router_testのテストで、routing tableを生成するのに、Router / Path::Pattern等の内部処理用クラスを直接使用してテストを行っていたのを、 publicなクラスであるMapperクラスを使用するよう修正しています。


move route allocation to a factory method on the mapping object

actionpack/lib/action_dispatch/journey/routes.rbactionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Mapperクラスに、Journey::Routeクラスのインスタンスを生成する為のファクトリーメソッドを追加しています。

Routeクラスのコンストラクタシグネチャを変更する予定がある為との事です。


Tiny documentation fixes [ci skip]

guides/source/testing.mdrailties/lib/rails/paths.rbの修正です。

guideに書いてあったexampleコードのシンタックスに誤りがあったのを修正、及び、:nodoc:を記載する箇所に誤りがあったのを修正しています。


Add a changelog entry for #21124 [ci skip]

railties/CHANGELOG.mdの修正です。

development環境ではI18n.load_pathを自動でリロードするよう対応した、Reload I18n locales in development の対応について、CHANGELOGに追記しています。