手数料を考慮して利用条件の判定を行うように修正

[chihiro-adachi]

概要(Overview・Refs Issue)

#4881 の対応

方針(Policy)

#4894 では、初期設定の支払方法がnullになるケースが有り、注文画面表示時に一部の決済プラグインでシステムエラーが発生する場合がありました。

初期設定の制御は従来どおりとして、注文画面での支払方法の一覧表示の制御のみ対応しています。

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

#4881 の再現手順の設定で検証しました。

• 配送方法設定から送料を0円で設定しておく
• 以下2点の支払い方法を追加
  • 代引(手数料：300円、利用条件：1〜1,399円)
  • 代引(手数料：0円、利用条件：1,400円〜)
• 販売価格：1,000円(税込1,100円)の商品を作成
• 上記の商品をカートにいれ、レジに進む
• 注文画面では、以下のように表示される

小計：￥1,100
手数料：￥0
送料：￥0
合計：￥1,100

支払方法：支払い方法もエラーも表示されない(内部的には「代引無料」が選択されている)

• 確認するを押下する

小計：￥1,100
手数料：￥0
送料：￥0
合計￥1,100

支払方法：支払い方法が表示されずエラーが表示される(内部的には未選択状態となる)

改善案

謎に購入フローが進めないという問題は解決していますが、購入はできず、本質的な問題解決は利用条件の設定方法を修正する必要があります。
ロジック変更で解決が難しいなら、管理画面の警告を工夫するのがいいかと思いました。
例えば上記の例ですと、登録時に

税込 1,100円 ~ 1,399円の購入で選択できる支払い方法がありませんがよろしいですか？

などの警告を出すか、あるいは

購入可能金額(税込): 300円 ~ 1,099円, 1,400円 ~

と金額が網羅できていないことがわかる表示を出せば、間接的ですが問題は解決すると思います。

[okazy]

問題となっていたケースで初期のお支払い方法がnullとなっていないことも確認しました。

[chihiro-adachi]

@okazy
配送方法設定時、選択した支払方法に利用できない金額帯が生じる場合、以下の警告を出力するように修正しています。

[chihiro-adachi]

@okazy @kiy0taka
警告表示の内容を詳細化しました。金額帯も出力するようにしています。