rails commit log流し読み(2017/03/11) - なるようになるブログ

2017/03/11分のコミットです。

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

Added #reverse_merge and #reverse_merge! methods to ActionController::Parameters

Make sure local cache cleared even it’s throwing:

activesupport/lib/active_support/cache/strategy/local_cache_middleware.rbの修正です。

ActiveSupport::Cache::Strategy::LocalCache moduleでエラー発生時のcache clear処理にrescue Exceptionを使用していたのを、ensureを使用して処理を行うよう修正しています。

+            cleanup_on_body_close = true
             response
           rescue Rack::Utils::InvalidParameterError
-            LocalCacheRegistry.set_cache_for(local_cache_key, nil)
             [400, {}, []]
-          rescue Exception
-            LocalCacheRegistry.set_cache_for(local_cache_key, nil)
-            raise
+          ensure
+            LocalCacheRegistry.set_cache_for(local_cache_key, nil) unless
+              cleanup_on_body_close
           end

wardenを使用している場合に、wardenがthrowされたエラーをcatchしてしまい、LocalCacheの後処理が行われないことがある為、確実に処理が行われるようにする為にensureを使用するようにしたとのことです。

Improve readability of testing guide [ci skip]

rails guideのA Guide to Testing Rails Applicationsの修正です。

Rails meets Minitestの項、testメソッドについて説明している箇所の言い回しを修正しています。

Merge pull request #28355 from Edouard-chin/reversemerge-for-strongparameters

actionpack/lib/action_controller/metal/strong_parameters.rbの修正です。

ActionController::Parametersクラスにreverse_merge及びreverse_merge!メソッドを追加しています。

Add @takkanm credits in the changelog [ci skip]

actionpack/CHANGELOG.mdの修正です。

先のreverse_merge、 reverse_merge!メソッドの対応についてのエントリーのクレジットにtakkanm さんを追加しています。元々同じ内容のPRをtakkanmさんが出されていた為。

Fix indentation and formatting in CHANGELOGs [ci skip]

actionpack/CHANGELOG.md、activesupport/CHANGELOG.mdの修正です。