Feature/phpstan level4 bugfixes

[chrisL-meier]

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更はありません
• フックポイントの呼び出しタイミングの変更はありません
• フックポイントのパラメータの削除・データ型の変更はありません
• twigファイルに渡しているパラメータの削除・データ型の変更はありません
• Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
• 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか
  • 権限を超えた操作が可能にならないか
  • 不要なファイルアップロードがないか
  • 外部へ公開されるファイルや機能の追加ではないか
  • テンプレートでのエスケープ漏れがないか

[sai-gillingham]

src/Eccube/Form/Type/Admin/OrderItemType.phpについて、

 ------ ---------------------------------------------------------------------------------------------- 
  Line   Eccube/Form/Type/Admin/OrderItemType.php                                                      
 ------ ---------------------------------------------------------------------------------------------- 
  229    Strict comparison using === between null and float|int|string will always evaluate to false.  
 ------ ---------------------------------------------------------------------------------------------- 

OrderItemTypeのgetPrice/setPrice/priceのPHPDoc@varにnullを追加してくれる？
OrderItemTypeのprice項目はNULL_OKではないけど、src/Eccube/Form/Type/Admin/OrderItemType.phpでOrderTypeのpriceはnullであれば、Productのpriceの値に変更しています。

[sai-gillingham]

コメント