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

2015/08/14分のコミットです。

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

descriptive error message when fixtures contian a missing column.

descriptive error message when fixtures contian a missing column.

activerecord/lib/active_record/connection_adapters/abstract/database_statements.rbの修正です。

fixturesに、テーブルに存在しないカラム名が指定されていた場合に明確にその旨わかるようエラーメッセージを表示するよう修正しています。

エラーメッセージ例。

# before
NoMethodError: undefined method `sql_type' for nil:NilClass

# after
ActiveRecord::Fixture::FixtureError: table "parrots" has no column named "arrr"

Merge pull request #19770 from vngrs/prevent_duplicated_where_clauses

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

abstract classでdefault scopeを使用していた場合に、default scopeに指定したwhereが2回設定されてしまうバグがあったのを修正しています。

例。

class User < ActiveRecord::Base
  self.abstract_class = true
  default_scope { where('deleted_at IS NULL') }
end

class Admin < User
end

Admin.all.load # => SELECT "admins".* FROM "admins" WHERE (deleted_at IS NULL) AND (deleted_at IS NULL)

4.2.0系から発生していたリグレッションのようです。

raise if anchor is passed to scope

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

scopeメソッドにanchorオプションが渡された場合に、ArgumentErrorをraiseするよう修正しています。

anchorオプションが指定された場合、matchメソッドの方でオプションの処理を行うため、scopeメソッドにわたってくる事は通常無いため、との事です。

Merge pull request #19683 from tristang/require-option-for-counter-cache

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

has_many associationに、counter_cacheオプションを指定出来るよう修正しています。

belongs_to associationにcounter_cacheオプションを指定してないにも関わらず、条件に一致するカラム名があった場合に自動でcounter cacheの処理が行われてしまう、というバグがあった為、has_many associationについても、明確にオプションが設定されてないとcounter cacheの処理が行われないよう対応しています。

pull anchor extraction up

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Mapperクラスのコンストラクタで行っていたanchorオプションの取得処理を、もっと早い段階(matchメソッド)で行うよう修正しています。

options hashの値を変化させる回数を減らす為との事。

remove anchor from mapping

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Mappingクラスからanchor変数を削除しています。

先のコミットでoptions hashからのanchorの取得処理はMapperクラスを呼ぶ前に行われるようになった為、そもそもMappingクラスでanchorの情報を保持しておく必要性がなくった為、削除したとの事です。

remove as

actionpack/lib/action_dispatch/routing/mapper.rbの修正です。

Mappingクラスからas変数を削除しています。

options hashからのasのオプションの値の取得処理はMapperクラスを呼ぶ前に行われるようになっており、Mappingクラスで情報を保持しておく必要性はなくなっている為、削除したとの事です。