なるようになるブログ

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

rails commit log流し読み(2017/10/22)

2017/10/22分のコミットです。

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


Avoid slicing from Thor's original HWIA

railties/lib/rails/generators/rails/plugin/plugin_generator.rbの修正です。

オプションのパース処理にsliceメソッドを使用していたのを、使用しないよう修正しています。

-      opts = (options || {}).slice(*PASSTHROUGH_OPTIONS)
+      opts = (options.dup || {}).keep_if {|k, | PASSTHROUGH_OPTIONS.map(&:to_s).include?(k) }

ここでのoptionsThor::CoreExt::HashWithIndifferentAccessクラスのインスタンスなのですが、Let Hash#slice return a Hashの影響(Hash#sliceの戻り値が必ずHashになった)で、sliceを使用すると戻り値がHashインスタンスになってしまい以降の処理でエラーになってしまう為、Hash#sliceは使用しないようにしています。


Merge pull request #29710 from padi/rails-ujs-docs

actionview/app/assets/javascripts/rails-ujs/utils/dom.coffeeactionview/app/assets/javascripts/rails-ujs/utils/event.coffeeのdocの修正です。

rails-ujsの、Rails.matchesRails.fire及びRails.delegateメソッドにdocを追加しています。