なるようになるブログ

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

rails commit log流し読み(2015/12/28)

2015/12/28分のコミットです。

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

activerecord/CHANGELOG.md


[ci skip] Fix typos in Active Support changelog.

activesupport/CHANGELOG.mdの修正です。

numericnumbericに、ActionControllerActiveControllerにそれぞれタイポしている箇所があったのを修正しています。


Removed extra "the" in the documentation of broadcast_message [ci skip]

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

Logger#broadcast_messagesオプションについてのdocのグラマーの修正(不要なtheの削除)を行っています。


tests, don't connect to mysql when running sqlite3 tests.

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

不正なconnectionを指定した場合のテストに、adapterがMysql2Adapterかどうかのチェックを追加しています。

元々、mysql2 adapterでしか動かないテストになっており(テストの前処理でmysql2 adapterを決め打ちしている)、sqlite3でテストを実行した場合にエラーになってしまっていたので、チェックを追加したとの事です。


Merge pull request #22053 from Empact/first-loaded

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

二つの対応が一つのPRでまとめて行われています。

まず一つ目は、ActiveRecord::FinderMethods#firstメソッドlimitオプションを指定した場合、かつ、対象のレコードが既にload済みの場合に、load済みの値を返すよう改善しています。

二つ目は、ActiveRecord::FinderMethods#find_nthメソッドoffset引数を渡すのがdeprecateになりました。offsetを指定したい場合、Relation#offsetメソッドの方を使ってね、との事です。