なるようになるブログ

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

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

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

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


Merge pull request #34893 from gmcgibbon/add_gotcha_note_to_helper_method_docs

actionpack/lib/action_controller/metal/helpers.rbのdocの修正です。

ActionController::Helpers.helpersのdocにrenderとはview contextが違う旨説明を追加しています。


add_reference/remove_reference takes keyword arguments

activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rbactiverecord/test/cases/migration/change_table_test.rbの修正です。

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


foreign_key takes keyword arguments

activerecord/lib/active_record/connection_adapters/sqlite3/schema_statements.rbactiverecord/lib/active_record/connection_adapters/sqlite3_adapter.rbの修正です。

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


create_table_definition takes keyword arguments

activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rbactiverecord/lib/active_record/connection_adapters/postgresql/schema_statements.rbの修正です。

create_table_definitionメソッドでキーワード引数を受け取れるよう引数の指定を修正しています。


Fix remaining keyword arguments warnings for Active Record

Active Record、Active Modelの修正です。

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


touch_attributes_with_time takes keyword arguments

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

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


Revert register proc takes keyword arguments

activemodel/lib/active_model/type/registry.rbの修正です。

Fix remaining keyword arguments warnings for Active Recordで行った、procのキーワード引数にdouble splatを指定した対応をRevertしています。

Ruby 2.7未満では空のキーワード引数が来た時にエラーになってしまう為。


Use String#+@ before mutating the result of Symbol#to_s

actionpack/lib/action_controller/metal/params_wrapper.rbactionpack/lib/action_controller/test_case.rbの修正です。

Symbol#to_sの戻り値を変更する前に、String#+@を使用してfreeze済みの場合は複製を使用するよう修正しています。Ruby 2.7でSymbol#to_sの戻り値がfrozen stringに変更になった為。


Fix index array's offset

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

Fix remaining keyword arguments warnings for Active Recordでキーワード引数とそれ以外を分離するようにした影響で参照するindexがずれていたのを修正しています。


Using a class method is not the preferred way to accept arguments for scopes

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

Passing in argumentsの項から、引数を受け取る場合クラスメソッドの方が好ましい方法である旨説明があったのを削除しています。

scopeの方がscopingと一緒に使用した場合に問題が少ない(クラスメソッドだと想定外の挙動になるケースがある)為。


Merge pull request #36907 from wjessop/string_attribute_should_compare_with_typecast_symbol_after_update

activemodel/lib/active_model/type/immutable_string.rbの修正です。

immutable stringのserialize処理でSymbolについても処理を行うよう修正しています。String型のカラムにSymbolを指定した場合に変更内容が正しく取得出来るようにする為。


Discard ActiveStorage::AnalyzeJobs on ActiveRecord::RecordNotFound

activestorage/app/jobs/active_storage/analyze_job.rbの修正です。

ActiveStorage::AnalyzeJobActiveRecord::RecordNotFoundが発生した場合に、jobを破棄するよう修正しています。ActiveRecord::RecordNotFoundが発生する場合、何らかの理由でjobが削除済みの為、リトライしても失敗する為。


Correct test class name

activestorage/test/jobs/analyze_job_test.rbの修正です。

テストクラス名がファイル名と一致してなかったのを修正しています。