不要かつ困惑的なFormTypeのCouponTypeのConstraintsの追加をやめる - Githubissues

EC-CUBE / coupon-plugin

GNU Lesser General Public License v2.1

11 stars 41 forks source link

不要かつ困惑的なFormTypeのCouponTypeのConstraintsの追加をやめる #154

Closed kazuyoshikakihara closed 2 years ago

kazuyoshikakihara commented 2 years ago

概要(Overview・Refs Issue)

CouponType の discount_rate のConstraintsについて。

最初にFormをbuildする際には

 new Assert\Range([
     'min' => 1,
     'max' => 100,
]),

と宣言されていますが、その後、POST_SUBMITのEventListenerの中で改めて

new Assert\Range([
    'min' => 0,
    'max' => 100,
]),

とmin値を変えてConstraints（validate）が追加宣言されています。

どちらのminが正しいのかプログラムからは判断がつきませんが、実質的に'min' => 1のほうが有効になっているので、POST_SUBMIT時の追加宣言は不要かつ困惑的と考え、削除します。

テスト（Test)

CouponType.php に対する自動テスト用のテストコードを新たに作成しました。

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

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

レビュワー確認項目

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

chihiro-adachi commented 2 years ago

@kazuyoshikakihara ありがとうございます！修正確認しました。