2014/12/03分のコミットです。
CHANGELOGへの追加はありませんでした。
Prefer object/nil over true
/false
activesupport/lib/active_support/message_verifier.rb
の修正です。
先日追加されたActiveSupport::MessageVerifier#verified
メソッドが、エラーが起きた際にnilを返すよう修正しています。
"true/false"を返すより、"object/nil"を返す方が判りやすく、使う方としても良いのでは、との事です。確かに、"object/false"を返すメソッドは基本的には見ないですよねえ。
Allow to unscope where conditions using arel_table
with Symbol
activerecord/lib/active_record/relation/query_methods.rb
の修正です。
unscope
+ where
の組み合わせを使用する際に、where
の条件にarel_table
+ Symbolを指定した際に正常に動作するよう修正しています。
ちょっとややこしいのですが、下記の用な使い方をしていた場合に、正常に動作していなかったとの事です。
DeveloperOrderedBySalary.where(Developer.arel_table[:name].eq('David')).unscope(where: :name).collect(&:name)
arel_table
にStringを指定した際は動作していたようです。
DeveloperOrderedBySalary.where(Developer.arel_table['name'].eq('David')).unscope(where: :name).collect(&:name)
Add some FIXME notes about documentation [ci skip]
activesupport/lib/active_support/message_verifier.rb
のdocの修正です。
各メソッドのコメントに、"FIXME: Document this method"を追加しています。
Refactor PredicateBuilder
from singleton to instance
ActiveRecordの修正です。
PredicateBuilder
クラスについて、singletonクラスで処理を行っていたのを、インスタンスを生成するように修正しています。
Arelからのtype cast処理を削除する為のリファクタリングの一貫との事です。
Clarify change to process helpers [ci skip]
rails guideのRuby on Rails 4.2 Release Notes
の修正です。
Action ViewのNotable changesの項に、integration testのprocessメソッドの仕様の変更について追記しています。
パスを指定する際に、先頭にスラッシュが必要になっています。
# OK get "/posts" # NG get "posts"
rails guideのRuby on Rails 4.2 Release Notes
の修正です。
先のコミットにタイポがあったのを修正しています。
Action View is needed for DebugExceptions
actionpack/lib/action_dispatch/middleware/debug_exceptions.rb
の修正です。
Action Viewのrequireを追加しています。DebugExceptions
クラスで必要な為との事。後で消す予定。
Refactor build_from_hash
to convert dot notation to hash first
activerecord/lib/active_record/relation/predicate_builder.rb
の修正です。
PredicateBuilder#build_from_hash
メソッドのリファクタリングです。最初にHashをドット表記に変換するよう修正しています。異なる形式のnested tableを同じ方法で扱えるようにする為、との事です。
require 'rake' before requiring Rake tasks
railties/lib/rails/tasks.rb
の修正です。
require 'rake'
を追加しています。処理の中でrake
のメソッドを使用しているのですが、requireを行うのが不足していた為。
Merge pull request #17888 from mrgilman/dot-notation
Require missing association in test
activerecord/test/cases/associations/has_one_through_associations_test.rb
の修正です。
不足していたrequire 'models/categorization'
を追加しています。
Fix "nonexistent" typo in tests
"nonexistant" -> "nonexistent"に修正しています。タイポ。
activerecord/lib/active_record/counter_cache.rb
の修正です。
使用していないactually_destroyed?
、clear_destroy_state
メソッドを削除しています。
[ci skip] remove references to Numeric#from_now
rails guideのActive Support Core Extensions
の修正です。
存在しないNumeric#from_now
メソッドについて説明している箇所を削除しています。
skip test to get sqlite3_mem
suite passing.
activerecord/test/cases/transactions_test.rb
の修正です。
in-memory database の場合、threadが死んだ時のテストをスキップするよう修正しています。