なるようになるブログ

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

rails commit log流し読み(2018/04/22)

2018/04/22分のコミットです。

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


Merge pull request #32514 from samdec/multiple-has-one-through-associations-build-bug

activerecord/lib/active_record/associations/has_one_through_association.rbの修正です。

has_one through associationが複数定義されている、かつ、そのassociationの中間オブジェクトが同じオブジェクトだった場合に、それらのassociationをそれぞれbuildした際、後にbuildしたの中間オブジェクトがbuildではなくsaveされてしまうバグがあったのを修正しています。


Using existing models for building multiple has_one through tests

Active Recordのテストの修正です。

先のhas_one throughのバグの修正で、テスト用のmodelを新規に作成していたのを、既存のmodelを使用するよう修正しています。


Merge pull request #32673 from eugeneius/raw_post_multiple_requests

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

ActionController::TestRequestクラスでrequest毎にRAW_POST_DATAをリセットするよう修正しています。


Fix typo in constant name [ci skip]

activesupport/CHANGELOG.mdの修正です。

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


Fix duplicated suffix for JobGenerator

activejob/lib/rails/generators/job/job_generator.rbrailties/lib/rails/generators/test_unit/job/job_generator.rbの修正です。

job generatorでjobを生成する際に、job名に_jobという接尾語がついていた場合、生成されるファイルに_jobが重複して指定されていまっていたのを、重複して指定されないよう修正しています。


Add test case to ChannelGeneratorTest

railties/test/generators/channel_generator_test.rbの修正です。

channel generatorでchannelを生成する際に、channel名に_channelという接尾語がついていた場合のテストを追加しています。


Merge pull request #32681 from bdewater/rubocop-0-54

rubocopのバージョンを0.54.0に更新しています。


Strip duplicated suffixes more strictly

各generatorで接尾語の重複を削除(job generatorなら_job)をする正規表現に、文字列の末尾指定(\z)を追加しています。元の正規表現だと文字列の途中に削除対象の文字があった場合にも削除してしまっていた為。


Merge pull request #31554 from fatkodima/controller_generator_suffix

railties/lib/rails/generators/rails/controller/controller_generator.rbの修正です。

controller generatorでcontrollerを生成する際に、controller名に_controllerという接尾語がついていた場合、生成されるファイルに_controllerが重複して指定されていまっていたのを、重複して指定されてないよう修正しています。


Fix indentation manually

railties/test/generators/app_generator_test.rbの修正です。

インデントがずれている箇所があったのを修正しています。