なるようになるブログ

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

rails commit log流し読み(2014/11/23)

2014/11/23分のコミットです。

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

activerecord/CHANGELOG.md


Fixing wrong link in 'Ruby on Rails Security Guide' [ci skip]

rails guideのRuby on Rails Security Guideの修正です。

CSRFの説明の箇所で、サンプルの例が誤っていたのを修正しています。

router reconfiguration by CSRFが正しいのでは、との事。


Remove outdated comments [ci skip]

actionpack/lib/action_dispatch/testing/assertions/routing.rbのコメントの修正です。

現在の仕様と異なる内容が記載されているコメントを削除しています。


Rename the primary key index when renaming a table in pg

activerecord/lib/active_record/connection_adapters/postgresql/schema_statements.rbの修正です。

PostgreSQLのadapterで、rename_tableを行った際、index名も合わせて変更するよう修正しています。


Fix build failures

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

先日、Fixture#findメソッドでunscopedを行う対応が行われたのですが、この影響でテストを同時実行した際にコケるテストが出てしまったので、unscopeにブロックを渡すよう修正しています。

-        model_class.unscoped.find(fixture[model_class.primary_key])
+        model_class.unscoped do
+          model_class.find(fixture[model_class.primary_key])
+        end

本来、他のテストに影響が出るのはおかしい筈なのですが、原因良く解らないので、とりあえずの対応をしたようです。


Anchor should not be appended when set to nil/false.

actionpack/lib/action_dispatch/http/url.rbの修正です。

add_anchorメソッドでanchorがnilの場合にフラグメントを追加しないよう修正しています。

rails 4.2でxxx_path、xxx_urlメソッドの引数にanchor: nilを指定した場合に、不要なフラグメント(#)が生成されてしまうバグがあったので、対応しています


Add changelog entry for .reflections API change

activerecord/CHANGELOG.mdの修正です。

rails 4.2からreflectionsメソッドがSymbolではなくStringを返すようになっているのですが、その対応についてCHANGELOGに記載が無かったので、追記しています。


Update reflections public API doc

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

reflectionsメソッドのサンプルの実行結果を修正しています。


Fix explanation for ActionDispatch::Callbacks.

rails guideのRails on Rackの修正です。

ActionDispatch::Callbacksについて、リクエストの処理前にprepare callbackが実行される旨説明が書かれていたのですが、 実際はprepare callbackは実行されないので、説明を修正しています。


[ci skip] Dispatching is a verb :bow:

rails guideのRails on Rackの修正です。

上記で修正したActionDispatch::Callbacksの説明のグラマーを修正しています。

-* Provide callbacks to be executed before and after the request dispatch.
+* Provides callbacks to be executed before and after dispatching the request.

"Dispatching"が動詞なので、との事です。


[ci skip] Fix comment of ActionDispatch::Callbacks

actionpack/lib/action_dispatch/middleware/callbacks.rbのdocの修正です。

上記guideの修正内容を、ActionDispatch::Callbacksのdocの方にも反映しています。


Test using ActionController::TestCase with engines

actionpack/test/controller/test_case_test.rbの修正です。

テストケースにrails engineとActionController::TestCaseを一緒に使用したテストを追加しています。


Correct example in the engines testing guide [ci skip]

rails guideのGetting Started with Enginesの修正です。

engineのテストについて説明したexampleに誤りがあったので、適切なexampleに修正しています。