なるようになるブログ

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

rails commit log流し読み(2017/09/28)

2017/09/28分のコミットです。

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

actionmailer/CHANGELOG.md


Merge pull request #30729 from yalab/nested_scaffold

railtiesの修正です。

scaffold_conrollerにnamespaceつきcontrollerを指定、かつ、model名を指定したような場合(e.g.g scaffold_controller admin/users --model-name=User`)に、デフォルトで生成されるcontroller及びviewファイルでリンクやリダイレクトで使用するリソース名に誤りがあったのを修正しています。


Merge pull request #30471 from dylanahsmith/remove-redundant-tracker

activerecord/lib/active_record/attribute_methods/dirty.rbactiverecord/lib/active_record/persistence.rbの修正です。

attributeが変化したかどうかを追跡するのに、変数を二つ(@mutation_tracker@mutations_from_database)使用していたのを、@mutations_from_databaseのみ使用するよう修正しています。

元々はchangesメソッドの古い挙動を追跡する為に二つ変数を用意していたのですが、masterブランチではchangesの古い挙動はもうサポートされておらず、@mutation_tracker@mutations_from_databaseが同じ内容を保持するようになった為、@mutation_trackerを削除しています。


Set version in activestorage/package.json in proper format.

activestorage/package.jsonの修正です。

version指定方法に誤りがあったのを修正しています。

-  "version": "5.2.0.alpha",
+  "version": "5.2.0-alpha",

npmだと元の書き方は出来ない為、とのことです。


Add assert_enqueued_email_with to ActionMailer::TestHelper

actionmailer/lib/action_mailer/test_helper.rbの修正です。

任意のmailがenqueueされかどうかを確認する為のassert_enqueued_email_withメソッドを追加しています。

assert_enqueued_email_with ContactMailer, :welcome, args: {email: 'user@example.com} do
  ContactMailer.with(email: 'user@example.com').welcome.deliver_later
end