なるようになるブログ

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

rails commit log流し読み(2020/08/27)

2020/08/27分のコミットです。

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

activesupport/CHANGELOG.md

actionview/CHANGELOG.md


Deprecate custom Action Mailer delivery job:

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

mail deliverに独自のjobを使用している場合に、そのjobのpeformメソッドのargs引数がキーワード引数で無い場合にdeprecateメッセージを表示するよう修正しています。

Revert "Remove deprecated ActionMailer::DeliveryJob and ActionMailer::Parameterized::DeliveryJobでRevertされたActionMailer::DeliveryJobActionMailer::Parameterized::DeliveryJobを削除する対応の関連で、左記対応がpeformメソッドのargs引数がキーワード引数である事を前提とする対応になってしまっていたのですが、MailDeliveryJobを継承していない、かつargs引数がキーワード引数でないjobがあった場合に、そのjobが正しく動作しなくなってしまうという問題が発生していました。そのため、左記のようなjobを使用している場合にdeprecateメッセージが出るように対応しています。


Implement #inspect for ActiveSupport::OrderedOptions

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

ActiveSupport::OrderedOptionsinspectメソッドを定義して、オブジェクトを表示する際に設定されている値が表示されるよう修正しています。

instance = ActiveSupport::OrderedOptions.new
instance.foo   = :bar
instance[:baz] = :quz
instance.inspect #=> "#<ActiveSupport::OrderedOptions {:foo=>:bar, :baz=>:quz}>"

Use super instead of self

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

`ActiveSupport::OrderedOptions#inspectで、selfの代わりに、superを使用するよう修正しています。その方がメソッド呼び出しが必要である旨がわかりやすい為、との事です。


Merge pull request #39420 from jonathanhefner/verba-sequentur-config

issueがsupport request / feature requestの場合に、自動でcloseするようbotの設定の追加を行っています。


Merge pull request #38699 from jasonyork/master

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

ActiveSupport::Durationin_seconds, in_minutes, in_hours, in_days, in_weeks, in_months, 及び in_yearsメソッドを追加しています。


Merge pull request #40114 from adrianna-chang-shopify/soft-deprecate-application-secrets

railties/lib/rails/application.rbrailties/lib/rails/commands/secrets/USAGEの修正です。

Rails::Application#secretsのdocを削除、及び、secretsコマンドを実行した際にdeprecateメッセージが表示されるよう修正しています。


Merge pull request #39350 from jaynetics/fix_rounding_of_custom_formatted_negative_amounts

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

number_to_currencyメソッドに負の値を指定した + custom formatを指定している場合に、値の丸め込みが正しく行われないバグがあったのを修正しています。

# Before
number_to_currency('-1,11') # => "$-1,00"

# After
number_to_currency('-1,11') # => "$-1,11"

Merge pull request #39363 from p8/format-did-you-mean-suggestions-on-error-pages

Action Packの修正です。

エラーページでDidYouMeanによるサジェスチョンを表示する際に、リストで値を表示するよう修正しています。


Merge pull request #39363 from seanpdoyle/translate-helper-yield-block

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

ActionView::Helpers#translateにblock指定出来るよう修正しています。blockには、translation keyとtranslationした値が渡されるようになっています。

<%= translate(".relative_key") do |translation, resolved_key| %>
  <span title="<%= resolved_key %>"><%= translation %></span>
<% end %>

Merge pull request #39337 from p8/did-you-mean-for-inverse-of-not-found

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

ActiveRecord::InverseOfAssociationNotFoundErrorのエラーメッセージで、DidYouMeanによるサジェスチョンを表示するよう修正しています。


Fix the test based on the changes on #40104

activerecord/test/cases/associations/inverse_associations_test.rbの修正です。

テストでdirtyhorrible等のネガティブな意味がある単語を使っていた箇所を、confusedpuzzled等に修正しています。