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

なるようになるブログ

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

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

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

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

actionview/CHANGELOG.md


Keep the value of $, and restore it

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

to_sentenceメソッドで$,の値を変化させているのですが、テストの後処理で元の値に戻すよう修正しています。


Merge pull request #27772 from y-yagi/remove_unreachable_code

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

AppGenerator.new及びPluginGenerator.newメソッドからapp_pathnilだった場合のチェック処理を削除しています。

app_pathはThorの必須パラメータとして定義されている、かつ、app_pathnilチェックより前にThorのパラメータ解析処理が行われており、app_pathnilの場合はそのパラメータ解析処理でエラーになりのチェックの時点でapp_pathnilになっている事は無い為。


Give a message to #test_duplicable assertion

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

duplicable?メソッドのテストで、assertionがエラーになった場合にテスト対象の値の詳細を表示するよう修正しています。


Change ActionView ERB Handler from Erubis to Erubi

Action Viewの修正です。

ERB Handlerで使用するgemをErubisからErubiに変更しています。

ErubiはErubisのフォークなのですが、現在もメンテされている、かつ、--enable-frozen-string-literalをサポートしており、性能的にErubisよりも良いため変更したとの事です。機能互換はある為、基本的にユーザ側は何も変更しなくて問題無いようになっています(一部対応していない機能あり)。また、Erubis handlerはdeprecateになり5.2で削除予定です。


This seems to be working on JRuby 9K

activesupport/test/inflector_test.rbの修正です。

parameterizeメソッドのテストがJRubyでskipされていたのを、JRubyでもテスト行うようskipを削除しています。

JRubyUTF-8からUTF8-MACへの変換が出来なかった為skipされていたのですが、現状問題なく変換処理が動作する為、テストを行うようにしたとの事です。


Add assertion messages

activerecord/test/cases/adapters/postgresql/infinity_test.rbactiverecord/test/cases/adapters/postgresql/numbers_test.rbの修正です。

assert_sendを使用していたのをassertを使用するよう修正した箇所で、assertion失敗時に詳細なエラーメッセージを表示するよう修正しています。


Fix grammar ‘an hyphen’ -> ‘a hyphen’ [ci skip]

actionview/lib/action_view/helpers/number_helper.rbactivesupport/lib/active_support/number_helper.rbメソッドのdocの修正です。

number_to_phoneメソッドのdoc内のグラマーの修正を行っています。