ぽかぽかウンティの自由帳

ぽかぽか動物園から投稿します。

Rails 本家にしたプルリクの変更が戻されたったww

マージしてくれたオーナーさんとは別のオーナーさんに突っ込まれて戻された。やっぱ影響範囲的にこわいなー。

github.com

しかし教養不足で英語がわからん。2行目の理解が多分違う。

I don't think this change is OK and is breaking Rails API.
我はこの変更を良しとしない。RailsAPI を壊している。

What if your controller does not include AC::Flash? we need to check for respond_to?(:flash) and it's the right thing to do there.
汝のコントローラーが ActionController::Flash を include しなかったらどうなる? たしかに flash メソッドの有無を確認する必要が出てくるだろう。

I'm reverting this until we find a proper fix for it. @supercaracal can you provide a sample app that reproduces the issue you're having? because I also feel that the provided tests are wrong.
我はこの問題の正しい解消方法を見付けられない限り、この変更を元に戻すことにする。この問題が再現できる汝のサンプルアプリケーションを我に提供できるか? なぜなら我はそのテストのやり方が誤っていると感じておるからだ。

どうしよう。英語が鬼門。

  • Redis Cluster を独自に session store として使っているから FW の session store は無効にしている
  • だから必然的に ActionDispatch::Flash Rack モジュールは使えなくなるので読み込んでいない
  • 元々 respond_to?(:flash) していたが、delegate 先で遅延評価されて NoMethodError を送出しているのはバグに思えた
  • そもそも Rails は session store を無効にしたり ActionDispatch::Flash Rack モジュールを読み込まなかったりするのを良しとしないつくりになっているのか?

的なことを英語で伝えたい...