なるようになるブログ

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

rails commit log流し読み(2017/04/04)

2017/04/04分のコミットです。

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


Revert “Merge pull request #28598 from wnadeau/patch-1”

forty_twoメソッドのdocのコメントを修正した、FinderMethods#fourty_two docs cite proper sourceをrevertしています。

-    # Find the forty-second record. Also known as accessing "the answer to life the universe and everything".
+    # Find the forty-second record. Also known as accessing "the reddit".

意図的に(ジョークとして)redditとしていたので、そのままの方が良いだろう、という事でrevertしたとの事です。この辺りのニュアンスはよくわからない…。


Merge pull request #28029 from koic/empty_insert_statement_value_not_supported

activerecord/test/cases/dup_test.rbactiverecord/test/cases/primary_keys_test.rbの修正です。

test_dup_without_primary_key及びtest_create_without_primary_key_no_extra_queryのテストを、Oracle adapterの場合実行しないよう修正しています。

Oracle adapterではempty_insert_statement_valueメソッドを実装していない(OracleINSERT .. DEFAULT VALUESが使えない)為、上記テストがエラーになってしまう為、実行しないようにしたとの事です。

詳細はこちらのコメント参照。


Merge pull request #28057 from eugeneius/clear_active_connections

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

Railsの初期化処理でactive connectionsをclearするよう修正しています。

初期化処理時に使用したconnectionsを再利用出来るようにする為(初期化処理とリクエストを処理するスレッドが異なる場合にconnectionsを使いまわせなくなってしまうのを避ける為)、との事です。


Merge pull request #27832 from bdewater/backtrace-silencer-regex

railties/lib/rails/backtrace_cleaner.rbの修正です。

backtrace cleanerで!~の代わりにRegexp#match?を使用するよう修正しています。