なるようになるブログ

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

rails commit log流し読み(2019/09/14)

2019/09/14分のコミットです。

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

activerecord/CHANGELOG.md

activejob/CHANGELOG.md


Merge pull request #37185 from seejohnrun/config-symbols

Active Recordの修正です。

database configurationのkeyが処理によってStringだったりSymbolだったりしていたのを、全てSymbolで扱うよう修正しています。

合わせて、keyがStringの値を返すようになっていたDatabaseConfig#configをDeprecateにし、keyがSymbolのconfigを返すDatabaseConfig#configuration_hashを追加しています。


Fix setting a more verbose thread-local log level

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

LoggerThreadSafeLevel#addメソッドでlog levelをチェックする際に、スレッドローカル変数ではなくインスタンス変数を参照するようになっていたのを、スレッドローカル変数を参照するよう修正しています。

これはそもそもRubyLoggerがlog levelをチェックする際にインスタンス変数を参照するようになっていた為で、サブクラスでlog levelを変更してもそれが使用されないようになっていました。

そのため、親クラスであるLoggerの処理を呼ばず、LoggerThreadSafeLevel側で処理全てを行うようにしています。因みに、Logger側にパッチを送って上記問題につていは修正済みとの事です。参考: Honor Logger#level overrides


Prefer assert_not_includes

activesupport/test/logger_test.rbの修正です。

assert_not + include?を使用していた箇所をassert_not_includesを使用するよう修正しています。


Move ActiveJob::LogSubscriber into its own file

Active Jobの修正です。

ActiveJob::Logging配下にあったLogSubscriberクラスをそれ単体のクラスに切り出しています。


Pass db_config object around instead of hashes

Active Recordの修正です。

connection specificationにconfigをHashではなくdb_configのobjectを渡すよう修正しています。

connection manager周辺でHashではなくdb_configのobjectで値を管理するよう予定しており、その一環との事です。


Stop queue_name_prefix from being global

activejob/lib/active_job/queue_name.rbの修正です。

job classでqueue_name_prefixを指定した場合に、その内容が他のjob classにも影響を与えてしまうようになっていたのを、その指定したjobでのみ指定した値が使用されるよう修正しています。


Merge pull request #37190 from glacials/fix_a_button_to_example

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

button_toメソッドのexampleでmethodオプションの指定方法に誤りがあったのを修正しています。


Merge pull request #37111 from okuramasafumi/latest-capistrano-in-asset-pipeline-guide

rails guideのThe Asset Pipelineの修正です。

Precompiling Assetsの項のCapistranoでデプロイする方法について説明している箇所で、Capistrano v3の場合の方法についての説明を追加しています。


IntegrationTest#head also does not accept non-keyword arguments

actionpack/lib/action_dispatch/testing/integration.rbの修正です。

IntegrationTest#headメソッドの引数指定を、キーワード引数の指定に修正しています。


Other get signatures take keyword arguments

actionpack/test/abstract_unit.rbの修正です。

getメソッドの引数指定にキーワード引数の指定を追加しています。


get gets keyworded options

actionpack/test/dispatch/session/cookie_store_test.rbの修正です。

こちらもテスト用のgetメソッドの引数指定にキーワード引数の指定を追加しています。


Reduce proc call

actionpack/lib/action_dispatch/middleware/cookies.rbの修正です。

CookieJar.buildメソッドから不要なproc callを削除しています。


Passing in a Hash instance as kwargs parameters requires the "double splat" prefix

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

Hashをキーワード引数として渡す際にdouble splatを指定するよう修正しています。


Fix config.cache_classes default value in guide.

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

config.cache_classesについての説明に、Springがインストールされているかどうかでデフォルトが変わる旨説明を追加しています。


tweaks in config.cache_classes docs [skip ci]

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

config.cache_classesについての説明を、クラスとモジュールに変更があった場合のみリロードされる旨説明を修正しています。