なるようになるブログ

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

rails commit log流し読み(2019/07/24)

2019/07/24分のコミットです。

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

activerecord/CHANGELOG.md


Merge pull request #36665 from jmschneider/master

activerecord/lib/active_record/connection_adapters/postgresql/oid/money.rbの修正です。

PostgreSQLのmoney columがcurrencyの指定が必須($12,345.67-$12.34は指定出来るが、12,345.67は指定出来なかった)になっていたのを、オプショナルになるよう修正しています。


Modify behaviour of AM::Errors#to_h:

activemodel/lib/active_model/errors.rbの修正です。

Deprecated AM::Errors#to_h:ActiveModel::Errors#to_hメソッドをdeprecateにして、to_hashメソッドを呼び出すだけの実装に変更したのですが、to_hto_hashは戻り値が異なる(to_hはエラーメッセージを一つだけ返すがto_hashは全てのエラーメッセージをは配列で返す)為、元の挙動を壊すようになってしまっていました。

そのため、to_hの元の挙動を維持するよう戻り値を調整するよう変更、及び、deprecateメッセージにto_hashto_hの戻り値の違いについての説明を含むよう修正しています。


Merge pull request #36370 from ptoomey3/master

activesupport/lib/active_support/parameter_filter.rbの修正です。

値の配列に対するparameter fileter処理で、filterに指定されたProcが実行されるようになっていたのかを、値の配列に対しても指定されたProcを実行するよう修正しています。


Fix broken rdoc for UrlHelper [ci skip]

actionview/lib/action_view/helpers/url_helper.rbのdocの修正です。

UrlHelper moduleの各メソッドのdocについて、不要なスペースによりフォーマットが壊れていた箇所をあったのを修正、及び、不要なクラスへのリンクがあったのを修正しています。


Fixed db:prepare task to not touch schema when dump_schema_after_migration is false.

activerecord/lib/active_record/railties/databases.rakeの修正です。

db:prepare taskがActiveRecord::Base.dump_schema_after_migrationの設定に関わらずschemaを生成するようになっていたのを、ActiveRecord::Base.dump_schema_after_migrationがfalseの場合はschemaを生成しないよう修正しています。


Merge pull request #36746 from abhaynikam/36738-followup-to-fix-typo-in-deprecation-message

activemodel/lib/active_model/errors.rbの修正です。

ErrorsErorsにタイポしている箇所があったのを修正しています。


Merge pull request #36741 from Edouard-chin/ec-system-test-url-options

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

Merge pull request #36691 from Edouard-chin/ec-system-test-routeで route helperをproxy classにdelegateするよう修正したのですが、その際に、url_optionsの再定義処理をproxy classに移動させるのが漏れていた(route helperを実行するclassでurl_optionsの再定義も行う必要があったのが、system test classで再定義したままになっていた)のを修正しています。