読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

rails commit log流し読み(2016/08/27)

2016/08/27分のコミットです。

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


Missing key should throw KeyError

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

ActionDispatch::Http::Headers#fetchに指定したkeyが存在しなかった場合に、KeyErrorをraiseするよう修正しています。

Rails 4.2系まではKeyErrorがraiseするようになっていたのですが、5.0.0でNameErrorをraiseするよう誤って変更してしまっていた為、修正しています。


Move custom assertion to its proper place

activesupport/lib/active_support/test_case.rbactivesupport/lib/active_support/testing/assertions.rbの修正です。

ActiveSupport::TestCaseクラスで定義されていたassert_nothing_raisedメソッドを、ActiveSupport::Testing::Assertions moduleに移動しています。他のRails独自のassertionメソッドはActiveSupport::Testing::Assertionsに定義されており、他と合わせる為assert_nothing_raisedも移動したとの事です。


Fix typo in deprecation message

activemodel/lib/active_model/errors.rbの修正です。

ActiveModel::Errors#add_on_blankメソッドを使用した場合に出力されるdeprecationメッセージで、代わりに使用すべきメソッド内にタイポがあったのを修正しています。

-          errors.add(attribute, :empty, options) if value.blank?
+          errors.add(attribute, :blank, options) if value.blank?

Fix typo in Delegation#delegate_missing_to doc [skip ci]

activesupport/lib/active_support/core_ext/module/delegation.rbのdocの修正です。

delegate_missing_toメソッドのdoc内にタイポがあったのを修正しています。