なるようになるブログ

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

rails commit log流し読み(2020/12/09)

2020/12/09分のコミットです。

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

railties/CHANGELOG.md

actionview/CHANGELOG.md


Fix schema dumper when datetime infitnity defaults are set

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

datetime columnのdefaultにinfitnityが指定されている場合に、defaultの値がschema.rbに正しい情報が出力されないバグがあったのを修正しています。


Merge pull request #40201 from Shopify/symbol-name

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

HashWithIndifferentAccess#convert_keyメソッドで、Symbol#nameメソッドが使用出来る場合(Ruby 3.0以上)に、Symbol#nameを使用するよう修正しています。 Stringオブジェクトの生成を減らす為。

参考: Feature #16150: Add a way to request a frozen string from to_s


Merge pull request #40760 from orhantoy/orhantoy/ignore-brewfile-lock

.gitignoreの修正です。

Brewfile.lock.jsonをignoreするよう修正しています。


Merge PR #40193

Style/RedundantRegexpEscape copを追加、及び、各ファイルにcopを適応しています。


Merge pull request #40754 from yahonda/revert_workarounds_for_ruby_17220

テスト用のworkerを作成する前にsleepするようにしたコミットをRevertしています。

getaddrinfo -> fork -> (forkしたプロセスで)getaddrinfo するとdeadlockが発生してしまうという問題の為のワークアラウンドだったのですが、Ruby側で問題が解決された為。


Merge pull request #40759 from orhantoy/broadcast-tagged-logging

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

ActiveSupport::TaggedLogging.newで元のloggerをdupしていたのを、cloneするよう修正しています。loggerに対してextendしていたメソッドを使えるようにする為。


Merge pull request #40734 from semaperepelitsa/benchmarkable

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

Rails moduleでActiveSupport::Benchmarkableをextendするよう修正しています。

benchmark method を、Rails.benchmark("test") { ... }のような形でどこでも使用出来るようにする為。


Fix regex for comments

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

Style/RedundantRegexpEscape copを追加した、Merge PR #40193でCOMMENT用のregexを誤って修正してしまっていたのを修正しています。


Call class method since sanitizer's instance method is private and add tests

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

rails-html-sanitizerの仕様変更により、sanitized_allowed_tagsメソッド、及び、sanitized_allowed_attributesメソッドが動作しなくなっていたのを修正しています。


Merge pull request #40766 from rafaelfranca/rm-queue-defaults

Rails 内部で使用しているbackground jobのqueue nameを全てjob adapterのデフォルトの値を使用するよう修正しています。

独自のqueue nameを使用すると、Sidekiqのようにqueue nameをconfigに指定する必要があるbackgroudの場合に、Railsのupgrade時にbackgroundのconfigも修正しなくてはならなくなってしまう為。


check yarn version and run correct install command

railties/lib/rails/tasks/yarn.rakeの修正です。

yarn v2の場合はyarn taskで実行するコマンドのオプションに--immutableを指定するよう修正しています。 yarn v2だと--no-progress --frozen-lockfileが使用出来ない為。


Merge pull request #38452 from jonathanhefner/config-action_view-image_loading

Action Viewの修正です。

image_tag:loadingオプションのデフォルト値を指定する為のconfig.action_view.image_loadingを追加しています。

imageのloadを全てlazyにしたい場合は、config.action_view.image_loading = "lazy"とすればOKです。


Add CHANGELOG entry for #38452

先のconfig.action_view.image_loadingの対応について、CHANGELOGのエントリーを追加しています。


Merge pull request #40096 from ldstudios/update-active-record-querying-guide

rails guideのActive Record Query Interfaceの修正です。

Specifying Conditions on the Joined Tablesの項に、Relation#mergeについての説明を追加しています。


Fixes the copy-paste typo for configuration guide changes made in PR: #40766

rails guideのConfiguring Rails Applicationsの修正です。

job名をタイポしていたのを修正しています。


Open transaction for multi-db apps

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

multiple databaseの指定がある場合、rails console --sandbox終了時にrollbackが行われないバグがあったのを修正しています。