なるようになるブログ

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

rails commit log流し読み(2020/01/20)

2020/01/20分のコミットです。

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


Fix typo

activerecord/test/cases/adapters/sqlite3/transaction_test.rbの修正です。

テスト名のタイポを修正しています。


Fix deprecate_methods to not expand positional argument hash

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

deprecate_methodsメソッドで、ruby2_keywordsを使用して、Hash引数の展開が行われないよう修正しています。


Extract message = options[method_name] out of method definition

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

メソッドが定義済み/未定義の場合それぞれで行っていたoptionsからの値の取得処理を、分岐処理前で行うよう修正しています。


add_foreign_key's arguments are compatible (invertable) for remove_foreign_key without customize

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

add_foreign_keyのinvert処理に独自のメソッド(invert_add_foreign_key)が使用されていたのを、remove_foreign_keyメソッドを使用するよう修正しています。add_foreign_keyremove_foreign_keyは同じオプションが指定出来、そのまま利用出来る為。


Invertable methods should have compatible method signature

Active Recordの修正です。

migrationのinvertable methodsの引数指定をキーワード引数で統一するよう修正しています。

invertable methodを実行する際にキーワード引数と非キーワード引数が混在してwarningが出てしまうのを避ける為。


*_for_alter methods should also takes keyword arguments

Active Recordの修正です。

先のキーワード引数の対応同様、*_for_alterメソッドもメソッドの引数指定をキーワード引数に修正しています。


Merge pull request #38260 from kamipo/fix_kwargs_warning_for_activejob

Action Mailer、Active Jobの修正です。

Active Jobでキーワード引数に関する警告が表示されないよう対応しています。