なるようになるブログ

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

rails commit log流し読み(2016/09/03)

2016/09/03分のコミットです。

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


Allow download of email attachments in the Mailer Preview template.

railties/lib/rails/templates/rails/mailers/email.html.erbの修正です。

mailer preview templateにattachmentsファイルダウンロード用のリンクを追加しています。

c8f0cb50-7126-11e6-846e-e9c28565c33d.png (561×363)


add Active Job bug report templates to guide [ci skip]

rails guideのContributing to Ruby on Railsの修正です。

先日追加されたActive Job用のbug report templatesファイルをCreate an Executable Test Caseの項に書いてあるbug report templatesの一覧に追加しています。


Fix broken heredoc indentation caused by rubocop auto-correct

normalizes indentation and whitespace across the project · rails/rails@80e66ccでrubocop auto-correctでフォーマットの修正を行ったのですが、その際heredocのインデントも削除されてしまっていました。

が、heredocについてはインデントがあった方が良いだろう、という事で再度インデントの調整を行っています。

        def non_kwarg_request_warning
           ActiveSupport::Deprecation.warn(<<-MSG.strip_heredoc)
-          ActionController::TestCase HTTP request methods will accept only
-          keyword arguments in future Rails versions.
+            ActionController::TestCase HTTP request methods will accept only
+            keyword arguments in future Rails versions.

-          Examples:
+            Examples:

-          get :show, params: { id: 1 }, session: { user_id: 1 }
-          process :update, method: :post, params: { id: 1 }
-        MSG
+            get :show, params: { id: 1 }, session: { user_id: 1 }
+            process :update, method: :post, params: { id: 1 }
+          MSG
         end

Remove blank else branch

activesupport/test/xml_mini/jdom_engine_test.rbの修正です。

JDOM engineのテストにあった不要なelse分岐を削除しています。


Extract duplicated create and create! definition for association

Active Recordの修正です。

ActiveRecord::Associations::CollectionAssociationActiveRecord::Associations::SingularAssociationそれぞれで同じ内容のcreatecreate!メソッドを定義していたので、共通の親クラスであるActiveRecord::Associations::Associationで定義し、子クラスで同じメソッドを定義しないよう修正しています。