nakagawa-takahiro / Assignment

MIT License
0 stars 0 forks source link

nullかどうかはisを使おう #18

Open yosukeaonuma opened 2 years ago

yosukeaonuma commented 2 years ago

https://github.com/nakagawa-takahiro/Assignment/blob/c4022b4c1750c032767a2d5edd761c937550dc73/fuel/app/classes/model/invite.php#L28

[コーディングスタイル] FuelPHPのクエリビルダーでは = でも中で勝手に is に変換して比較してくれているのだけど、実際に動いているSQLは IS NULLとなっているので、 かんりくんの開発ではコード上では('checked_at', 'is', null)のように書くようにしている! (is, nullを小文字にするとかも規約で基本的には統一している)

nakagawa-takahiro commented 2 years ago

対応済みです!