[4.2] ポイントがマイナスの場合、受注を更新できない不具合の修正

[LeTanNghia]

概要(Overview・Refs Issue)

以下のissue #5795 の対応です。
[4.2] ポイントがマイナスの場合、受注を更新できない

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

この問題を修正し、次のような機能テストを行いました。

ユーザーのポイントはプラスです

Case 1: 旧請求書と新請求書のポイント差額 1500 - 1000 = 500
Step 1: ユーザーズポイント:1000

Step 2: ご注文のポイント:1000

Step 3: 入力点: 1500
結果: 正常に更新されました

Case 2: 旧請求書と新請求書のポイント差額 2000 - 500 = 1500
Step 1: ユーザーズポイント:500

Step 2: ご注文のポイント:500

Step 3: 入力点: 2000
結果: 更新に失敗しました

Case 3:旧請求書と新請求書のポイント差額 500 - 1000 = -500
Step 1: ユーザーズポイント:500

Step 2: ご注文のポイント:1000

Step 3: 入力点: 500
結果: 正常に更新されました

Case 4: 旧請求書と新請求書のポイント差額 1000 - 1000 = 0
Step 1: ユーザーズポイント:500

Step 2: ご注文のポイント:1000

Step 3: 入力点: 1000
結果: 正常に更新されました

ユーザーのポイントは否定的です。
Case 5: 旧請求書と新請求書のポイント差額 1000 - 500 = 500
Step 1: ユーザーズポイント: - 500

Step 2: ご注文のポイント: 500

Step 3: 入力点: 1000
結果: 更新に失敗しました

Case 6: 旧請求書と新請求書のポイント差額 500 - 1000 = - 500
Step 1: ユーザーズポイント: - 500

Step 2: ご注文のポイント:1000

Step 3: 入力点: 500
結果: 正常に更新されました

Case 7: 旧請求書と新請求書のポイント差額 1000 - 1000 = 0
結果: 正常に更新されました
Step 1: ユーザーズポイント: - 500

Step 2: ご注文のポイント:1000

Step 3: 入力点: 1000

Unit test:

相談（Discussion）

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

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

レビュワー確認項目

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

[chihiro-adachi]

@LeTanNghia
$diffUsePointはマイナスの場合もあるので、条件の見直しをお願いします。

[chihiro-adachi]

マイナスの場合は会員の所持ポイントに戻す場合なので、この条件で問題なさそうです。失礼しました。

[chihiro-adachi]

ご対応ありがとうございます