なるようになるブログ

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

rails commit log流し読み(2021/08/18)

2021/08/18分のコミットです。

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


Avoid orders_count in Active Record order guide examples

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

orderメソッドのexampleでorders_count を使用しないよう修正しています。orders_countorderと関係あるのでは、と勘違いさせてしまう可能性があるため、とのことです。


Do not change the framework if all we want are configs

actionpack/lib/action_controller/railtie.rbactivejob/lib/active_job/railtie.rbの修正です。

code内で使用していないattrをbase classに追加しないよう修正しています。


Document query tags configs

rails guideのConfiguring Rails Applicationの修正です

query tags関連のconfigについての説明を追加しています。


Avoid using class_eval and just include a module

query tagの機能を追加するのにclass_evalを使用していたのに、関連する機能をmoduleに切り出して単純にincludeするだけに修正しています。


Some stylistics changes

Active Recordの修正です。

コードのスタイル(<<を2回使っていた箇所をArrayを+するよう修正など)の修正を行っています。


Remove more unneeded methods that should be configs

Active Recordの修正です。

configの設定をする時だけ必要な値に対する不要なメソッドを削除しています。


Move default tagging configuration to the railtie

activerecord/lib/active_record/query_logs.rbactiverecord/lib/active_record/railtie.rbの修正です。

デフォルトのtaggingの設定をrailtiesで行うよう修正しています。他のconfigと同じ箇所でデフォルト値を設定するようにする為。


No need for memoization

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

アプリケーション名に関する不要なメモ化を削除しています。


Allow tag values to be static

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

tagにstaticな値を指定出来るよう修正しています。


Fix initialization proccess of the query tags

actionpack/lib/action_controller/railtie.rbactivejob/lib/active_job/railtie.rbの修正です。

query tags関連の処理で、configに指定した値が正しく反映されてなかったのを修正しています。


Improve taggings API by introducing a null object

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

taggings contextが未設定かどうかを扱うのにnilではなく、専用のクラス(NullObject)を使うよう修正しています。&.をchainしないで済むようにする為。


Stop relying on mutating the configuration object

actionpack/lib/action_controller/railtie.rbactivejob/lib/active_job/railtie.rbの修正です。

configuration objectの値を設定処理中に変更しないよう修正しています。


Only try to configure the query log tags if Active Record is present

actionpack/lib/action_controller/railtie.rbactivejob/lib/active_job/railtie.rbの修正です。

Active Recordのconfigが定義済みの時のみ、config.active_record.query_log_tags_enabledを参照するよう修正しています。


No need to check if the logs are enable to run those tests

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

不要なlog tagsが有効化されているかどうかのチェックを削除しています。


Fix action_on_unpermitted_parameters setter

actionpack/lib/action_controller/railtie.rbの修正です。

action_on_unpermitted_parametersにfalseが指定されていた場合に、その指定が無視されてしまっていたのを修正しています。


Delay the exclusion of the configs to the last minute

actionpack/lib/action_controller/railtie.rbactivejob/lib/active_job/railtie.rbactiverecord/lib/active_record/railtie.rbの修正です。

まだ使用される可能性がある前にconfigの除外処理をしてしまっていたのを、除外処理を実際に必要になる直前で行うよう修正しています。


Tweaks wording in guide [skip ci]

rails guideのAutoloading and Reloading Constantsの修正です。

Autoloading and Enginesの項の言い回しを修正しています。