Rails6でadd_referenceすると `null: false` が追加されるのだが
タイトルの通り、rails g migration AddColumnsToTable name user:references を実行すろと、null: falseオプションが追加される仕様に異議を唱えるissue。
この人が言っていることは2つあって
- null: falseつくけど、
belongs_toがオプショナルな可能性もあるじゃん？ - 既存テーブルにadd_referenceのままを適用したら defailt値に nullが入ってしまいPostgreSQLだとエラーになるよ。null: falseいらなくない？
と言っている。
このような振る舞いになったプルリクエストはコメントで貼られてissueはクローズされそうなムード。
ここからは僕の感想なんですが、DBに適用するのはお前の仕事じゃん。generatorはただのgeneratorなんだからお前の用途に合わせてお前が実行前に修正しろ。
- as for
- ~に関して
パーシャルのコレクションレンダリングでnilを渡す
パーシャルをレンダリングする時ってeachで回すと、N+1クエリ状態ですごく遅いんだよね。そういう時はパーシャルのレンダリングオプションにcollectionをつけることで、 レンダリングのオーバーヘッドを抑えることができる。
render 'books', as: :book, collection: books
で、このissueは render 'books', as: :book, collection: nil だと、エラーが起きていない仕様に対して意義を唱えている。
インスタンス変数って未定義だとnilじゃん。タイポがあるとエラーの発見できないじゃん。だからcollectionにnilが入ってきたらエラーを投げようぜ。ということが書いている。
このissue主はプルリクエストも作っていて、そのプルリクエストには破壊的変更なので非推奨から起こす必要がたぶんあるってコメントをもらっている。
僕自身はこの変更についてはどっちでもいいかなと思う。rubocopで検出できる内容だし。
number_to_currencyの出力に半角スペースではなくnbspを使ってくれ
半角スペースを含む€のような通貨だと意図しない改行が発生してしまう、という文脈みたい。 ちなみに、CSSにはスペースなどの単語の区切りで改行するようになっている。
今までは半角スペースを置換するパッチが活躍していたらしい。
HTMLEntities.new.encode(Class.new.extend(ActionView::Helpers::NumberHelper).number_to_currency(12, locale: 'de').gsub(' ',"\u00A0"), :named)
=> "12,00 €"
- be addressed
- 問題に取り組む
link_toでXSS
link_to の第2引数に javascript:...を渡すとそのまま発火するよ。というissue。
本文には対策のdiffも添えられている。
- dofer to~
- 〜に従う
- familiar with
- ~に精通
ActiveRecord#valid?はinteger型上限バリデーションを含めるべき rails/rails#37256
validationを設定していないモデルのテーブルにint(11)型で定義したカラムにint(11)を超える値を設定すると、バリデーションエラーが起きないで、例外が投げられる挙動について書かかれたissue。
このissueについたコメントは、int(11)の範囲内を保証するvalidationつければいいんじゃねえの？という真っ当なコメント。 このコメントに対して、validationを設定したら他のvalidationとレイヤーが違うでしょ。だからデフォルトのvalidationに追加が必要だと思う、と返信をした。
所感
デフォルトバリデーションが具体的にどんな振る舞いをするのかはわからないので、それ次第ですが、 想定外の値が入ってくることは不具合の可能性が高いので、validationをすっ飛ばして例外を投げている現在の実装に対して不自然さは特に感じない。
Migrationが2回走る #37225
Migration running twice #37225
Migration running twice · Issue #37225 · rails/rails · GitHub
質問1
railsコマンドを実行すると2回実行してしまう。一体何が起きているんですか？ (本文に再現コードあり)
回答1
This file does not have the migration code in an up method.
そのファイルにはマイグレーションファイルの中にupメソッドがない。
I would venture a guess that the file is getting run multiple times and running the defined migrations each time
そのファイルは複数回実行されて、マイグレーションの定義を毎回実行する、と検討をつけるでしょう。
The syntax highlighting is a dead giveaway, watch out for that 😎
シンタックスハイライトは死んでる。注意してね。
質問2
Yeah, that would be it. Why isn't that an error? "Migration has no up or change method"
おおおお、たぶんそれっぽい。なぜマイグレーションにup か changeが無い時にエラーが起きないの？
回答3
Yeah it's just a ruby file and those methods (create_table, add_index, etc) are class methods.
それはただのrubyファイルであり、クラスメソッドです。
It's definitely a gotcha but unfortunately it's valid ruby.
たしかに落とし穴(?)しかし、残念ながら正しいRubyです。
When you require the file, it will perform the method calls just as directed.
そのファイルが必要な時、そのメソッドを直接実行するでしょう。
Also, up, down, and change are all intentionally optional.
また、up, down, changeはすべて意図的にオプションです。
This allows you to do things like comment out all your methods to force a migration to be marked as ran.
あなたのメソッドをコメントアウトするように、マイグレーションを強制的にマークすることができるようになります。
4.2の互換性を断つなら、down, up, change以外でのDSLをブロックできる、ということが書いていた。 おわり。
Rails migration guide does not mention the removal of render :text #37145
render :text が削除されたことがアップグレード移行ガイドにに書かれていない、という指摘のissueだ。
本文
render :text was removed in 5.1 but it is not mentioned in the upgrading guide for 5.1.
render :textが5.1から削除されたが、アップグレード移行ガイドに触れられていない。
It is mentioned that it is discouraged under the 4.0 to 4.1 upgrade but it would be useful to see it mentioned on the release it was removed from.
4.0から4.1のアップグレードで非推奨だと記載していたが、削除されたリリースで言及されていると便利です。
なお、本issueへのプルリクが作られているけど、本件についてリリースノートで書かれているよ、というツッコミがきていた。
