rails commit log流し読み(2015/10/27) - なるようになるブログ

2015/10/27分のコミットです。

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

[ci skip] Fix method documentation location

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

Preloader#preloadメソッドの実装とdocの間にNULL_RELATION定数があったのを、定数の位置を動かして、実装とdocが連続するよう修正しています。

Merge pull request #19924 from iamvery/db-tasks-exit-status

activerecord/lib/active_record/tasks/database_tasks.rb、 activerecord/lib/active_record/tasks/sqlite_database_tasks.rbの修正です。

DBのcreate / drop用rakeタスクで、処理が失敗した場合に、exit statusに0ではなく1を設定するよう修正してます。

&&で複数のコマンドまとめて実行する(bin/rake db:setup && bin/rspecとか)際に、exit statusが0だと正常と判定されて不味いんですねえ。

Edits default config/boot.rb to match what's generated by Rails 4.2.

rails guideのThe Rails Initialization Processの修正です。

guideに記載されているconfig/boot.rbの内容を、実際生成されるconfig/boot.rbの内容と合わせるよう修正しています。

Merge pull request #19686 from tsun1215/index_errors

activerecord/lib/active_record/associations/builder/has_many.rb、 activerecord/lib/active_record/autosave_association.rbの修正です。

nested attributesでエラーになった際に、エラーメッセージにインデックスを表示出来るよう対応しています。

例。

class Guitar < ActiveRecord::Base
  has_many :tuning_pegs, index_errors: true
  accepts_nested_attributes_for :tuning_pegs
end

class TuningPeg < ActiveRecord::Base
  belongs_to :guitar
  validates_numericality_of :pitch
end

# 元々のエラーメッセージ(インデックス無し)
`guitar.errors["tuning_pegs.pitch"] = ["is not a number"]`

# 対応後(インデックスあり)
`guitar.errors["tuning_pegs[1].pitch"] = ["is not a number"]`

インデックスの表示はデフォルトではfalseになっており、associationsにindex_errorsオプションを指定するか、または、グローバルのconfig.active_record.index_nested_attribute_errorsにtrueを設定すると表示されるようになります。

[ci skip] Add doc to preloaders_on

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

Preloader#preloaders_onメソッドにdocを追加しています。

[ci skip] Clarify doc for preloaders_for_one

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

Preloader#preloaders_for_oneメソッドのdocにロード処理についての説明、及び各preload classのrunメソッドが呼ばれる旨説明を追加しています。

[ci skip] Add doc for preloader_for

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

Preloader#preloaders_forメソッドにdocを追加しています。

Merge pull request #21994 from mtodd/inherit-scopes

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

scopeの継承についてのテストが正しく行われていなかった(ActiveRecord::Relationはlazily loadの為、実際の処理を呼び出さないと評価されなかった)のを修正しています。