なるようになるブログ

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

rails commit log流し読み(2018/05/23)

2018/05/23分のコミットです。

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


Merge pull request #32931 from jeremyevans/fast-xor

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

ActionController::RequestForgeryProtection#xor_byte_stringsメソッドをリファクタリングしています。

ループ数やpackメソッドの呼び出しを削減し、1.7倍程度高速になっているとの事です。


Merge pull request #32934 from aki77/fix-locale-selector

railties/lib/rails/templates/rails/mailers/email.html.erbの修正です。

email previewでlocaleを変更した際に、subjectが変更されないバグがあったのを修正しています。


Remove 7 years old note-to-self by tenderlove

actionpack/lib/action_dispatch/journey/nfa/simulator.rbのdocの修正です。

Simulator#simulateメソッドから不要なコメントを削除しています。


Rollback parent transaction when children fails to update (#32796)

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

autosave associationsを使用している場合に、子のassociationsの保存に失敗した場合、親associationのtransactionをrollbackするよう修正しています。


Bump minimum version of Thor to 0.19.0

railties/railties.gemspecの修正です。

dependencyに指定するThorのminimum versionの指定を0.18.1から0.19.0に更新しています。

Thor 0.18だとfrozen string literalを指定している場合に、コマンド実行時にFrozenErrorが発生してしまうという問題がある為。


Indicate true in a doc comment is code-like.

actionview/lib/action_view/helpers/cache_helper.rbのdocの修正です。

CacheHelper#cache_fragment_nameメソッドのdoc内のskip_digestオプションについて説明している箇所で、オプションのkeyだけが等幅フォントで表示されていたのを、value等幅フォントで表示されるよう修正しています。


Merge pull request #32144 from kazu9su/master

activestorage/test/service/disk_service_test.rbの修正です。

ActiveStorage::Service::DiskService#url_for_direct_uploadメソッドについてのテストを追加しています。


Merge pull request #32851 from yskkin/doc_require_dependency

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

require_dependencyメソッドがAPI docに表示されるよう修正しています。

require_dependencyは既にRails guideで紹介されており、publc API扱いの為。


Merge pull request #32699 from printercu/patch-3

railties/lib/rails/tasks/yarn.rakeの修正です。

yarn:install taskで./bin/yarnを実行する際に、NODE_ENVを指定するよう修正しています。

環境変数NODE_ENVが指定されていればその値を、なければRAILS_ENVを見に行くようになっています。


Merge pull request #32822 from lxxxvi/improved_error_message_in_assert_changes

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

assert_changesメソッドでassertionでエラーになった際のメッセージを修正しています。

expected / actualが明確にわかるようになっています。

# before
"value" didn't change to y

# after
"value" didn't change to as expected
Expected: "y"
  Actual: "z"

Fix RDoc formatting: + doesn't work with space [ci skip]

actionview/lib/action_view/helpers/cache_helper.rbのdocの修正です。

CacheHelper#cache_fragment_nameメソッドのdoc内のskip_digestオプションが正しく等幅フォントで表示されていなかったのを修正しています。