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.rb
、
activerecord/test/cases/migration/change_table_test.rb
の修正です。
キーワード引数にdouble splatを指定するよう修正しています。
foreign_key takes keyword arguments
activerecord/lib/active_record/connection_adapters/sqlite3/schema_statements.rb
、
activerecord/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.rb
、
activerecord/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.rb
、
actionpack/lib/action_controller/test_case.rb
の修正です。
Symbol#to_s
の戻り値を変更する前に、String#+@
を使用してfreeze済みの場合は複製を使用するよう修正しています。Ruby 2.7でSymbol#to_s
の戻り値がfrozen stringに変更になった為。
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
と一緒に使用した場合に問題が少ない(クラスメソッドだと想定外の挙動になるケースがある)為。
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::AnalyzeJob
でActiveRecord::RecordNotFound
が発生した場合に、jobを破棄するよう修正しています。ActiveRecord::RecordNotFound
が発生する場合、何らかの理由でjobが削除済みの為、リトライしても失敗する為。
activestorage/test/jobs/analyze_job_test.rb
の修正です。
テストクラス名がファイル名と一致してなかったのを修正しています。