activerecord/lib/active_record/fixture_set/model_metadata.rb、 activerecord/lib/active_record/fixture_set/table_rows.rb、 activerecord/lib/active_record/fixtures.rbの修正です。

ActiveRecord::FixtureSet::ModelMetadata.newの引数から不要なtable_name引数を削除しています。

table_nameはmodel_class引数が指定された場合のみ使用されているのですが、model_classが渡された場合、table_nameはmodel_classから取得出来る(model_class.table_name)為。

Move resolve_sti_reflections which is table row related code into TableRow

activerecord/lib/active_record/fixture_set/table_row.rb、 activerecord/lib/active_record/fixture_set/table_rows.rbの修正です。

resolve_sti_reflectionsメソッドをActiveRecord::FixtureSet::TableRowsからActiveRecord::FixtureSet::TableRowに移動しています。rowsではなく、rowに関する処理の為。

2018-11-03

rails commit log流し読み(2018/11/02)

2018/11/02分のコミットです。

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

activesupport/CHANGELOG.md

Fix bug where #to_options for ActiveSupport::HashWithIndifferentAccess would not act as alias for #symbolize_keys.

Make #to_options an alias for #symbolize_keys

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

HashWithIndifferentAccessに、symbolize_keysのaliasとしてto_optionsメソッドを定義しています。

Rails 5.1まではHashWithIndifferentAccess#to_optionsはsymbolize_keysのaliasとして動作していたのですが、5.2でHashWithIndifferentAccessにsymbolize_keysメソッドを独自に定義した影響で、同じ振る舞いになっていませんでした。

で、5.1までの挙動と合わせる為、symbolize_keysのaliasとして明示的にto_optionsメソッドを定義するようにしています。

2018-11-02

rails commit log流し読み(2018/11/01)

2018/11/01分のコミットです。

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

Make i18n locale setting docs use around_action

rails guideのRails Internationalization (I18n) APIの修正です。

localeをcontrollerで変更するexampleがbefore_actionでI18n.locale=を使うexampleになっていたのを、around_action + I18n.with_localeを使うexampleに修正しています。

元々はI18n.locale=はスレッドセーフではないので、と理由で修正が始まったのですが、それは誤り(I18n.locale=はスレッドセーフ)でした。ただ、それを抜きにしても、処理が終わったら元のlocaleに戻した方が良いのでは、という意見があり、around_action + I18n.with_localeを使う形(actionの処理後に元のlocaleに戻る形)に修正しています。

Updating sample code on ActiveRecord#before_destroy callback [ci skip]

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

ActiveRecord::Callbacks moduleのdoc内、before_destroyのexampleでdelete_allの引数の指定の仕方が誤っていたのを修正しています。

2018-11-01

rails commit log流し読み(2018/10/31)

2018/10/31分のコミットです。

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

activerecord/CHANGELOG.md

update_columns now correctly raises ActiveModel::MissingAttributeError if the attribute does not exist.

activejob/CHANGELOG.md

Restore HashWithIndifferentAccess support to ActiveJob::Arguments.deserialize.

Merge pull request #34314 from bf4/patch-2

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

ActionController::API classのdocに、API Controllerはcookies, sessionsが使用出来る旨説明があったのですが、実際は使用出来ないので該当部分を削除しています。

update_columns raises if the column is unknown

activemodel/lib/active_model/attribute.rb、 activerecord/lib/active_record/persistence.rbの修正です。

update_columnsに存在しないカラムを指定した場合に、ActiveModel::MissingAttributeErrorをraiseするよう修正しています。

元々はRubyレベルではチェック出来ておらず、存在しないカラムを指定した場合、query実行時にエラー(no such columnでエラー)になっていました。

が、これだとignored_columnsに指定しているカラムも更新出来てしまう為、Rubyレベルで正しくチェックするよう修正しています。これにより、元々update_columnsでignored_columnsに指定していたカラムも更新出来てしまっていたのが、更新出来なくなっています。

Fix failing test

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

先のupdate_columnsの修正で、updateクエリ実行前にattributeの割当を行うようになったのですが、それだと更新時に使用するidも変わってしまう可能性がある為、元のidを処理前に保持して、更新処理は同じidが使用されるよう修正しています。

Merge pull request #33972 from bogdanvlviv/follow-up-33897

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

enqueue_retry.active_jobで表示するエラーメッセージで、original exceptionが取得出来ない場合、original exceptionに関するメッセージを表示しないよう修正しています。

Update authorization.rb

actioncable/lib/action_cable/connection/authorization.rbのdocの修正です。

Authorization#reject_unauthorized_connectionメソッドのdocから不要なエスケープを削除しています。

Fix example for database-specific locking clause

activerecord/lib/active_record/locking/pessimistic.rbのdocの修正です。

ActiveRecord::Locking::Pessimistic moduleのdoc内、lockメソッドのexampleにDB固有のロック処理(FOR UPDATE NOWAIT)を使用するよう修正しています。

直前の説明でlockにはDB固有のロック処理を使用出来る旨説明しており、その説明と合わせる為。

Restore HWIA support to AJ::Arguments.deserialize

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

ActiveJob::Arguments.deserializeメソッドにkeyがSymbolになっているHashWithIndifferentAccessを指定した場合に、keyが全てStringになってしまっていたのを、元の状態(keyがSymbolの場合はSymbol)を維持するよう修正しています。

Fix "warning: shadowing outer local variable - role"

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

warning: shadowing outer local variableのwarningが出ていたのを修正しています。

なるようになるブログ

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

rails commit log流し読み(2018/11/06)

rails commit log流し読み(2018/11/05)

rails commit log流し読み(2018/11/04)

rails commit log流し読み(2018/11/03)

rails commit log流し読み(2018/11/02)

rails commit log流し読み(2018/11/01)

rails commit log流し読み(2018/10/31)