なるようになるブログ

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

rails commit log流し読み(2017/01/12)

2017/01/12分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activesupport/CHANGELOG.md


remove unused requires

railtiesの各ファイルから不要なrequireを削除しています。


Revert "Merge pull request #27586 from maclover7/jm-fix-27584"

ActionController::TestCaseクラスを使用している場合にget以外のhttp request用メソッドでcookiesの更新がされないバグがあったのを修正したUpdate cookies helper on all HTTP requestsをrevertしています。

controllerでcookieを設定した場合のテストの挙動でリグレッションが発生した為、との事です。


Merge pull request #27610 from Envek/fix_and_speed_up_duration_parsing

Active Supportの修正です。

durationsにyear、又はmonthを指定した場合にparse結果の値が一致しないケースがあったのを、一致するよう修正しています。

# before
1.year == 12.months
# => false

# after
1.year == 12.months
# => true

"1 month"、及び"1 year"として扱うデータに不一致があったのが問題だったようです。


beginrescueend

activerecord/lib/active_record/coders/yaml_column.rbの修正です。

check_arity_of_constructorメソッドから不要なbegin句を削除しています。メソッド全体に対して例外補足を行う場合、rescuだけがあれば大丈夫な為。


Reduce string objects by using \ instead of + or << for concatenating strings

長い文字列を生成するのに+, <<メソッドを使用していたのを、\を使用し不要なString objectの生成を行わないよう修正しています。


Add additional tests for #27610

activesupport/test/core_ext/duration_test.rbの修正です。

durationsにyear又はmonthを指定した場合にparse結果の値が一致しないケースがあったのを修正した、Fix inconsistent results when parsing large durations and constructing durations from codeの対応について、テストを追加しています。


Add duration constructors for use in Numeric extensions

Active Supportの修正です。

durationを生成するようのNumeric methods(e.g. 1.day, 1.month等)でActiveSupport::Durationの定数を直接参照しデータを作成していたのを、ActiveSupport::Durationにduration生成用のラッパーメソッドを作成し、そちらを使用するよう修正しています。Numeric methods側でActiveSupport::Durationの内部実装を意識しないで良いようにする為。


fix method name in Renderer doc [ci skip]

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

ActionController::Renderer#newメソッドのdoc内に既に存在しないメソッドについての説明があったのを修正しています。