不要な分岐をなくす

[tatsuishi]

概要(Overview・Refs Issue)

不要な分岐をなくす

方針(Policy)

src/Eccube/Resource/template/admin/Setting/Shop/payment_edit.twigに
必要のないif文の分岐があったため削除しました。

実装に関する補足(Appendix)

PaymentController.phpのoldPaymentImageは必要である可能性も考え、削除していません。

テスト（Test)

相談（Discussion）

PaymentController.phpのoldPaymentImageは、プラグインでの使用はあるのでしょうか?
現状は削除していません。

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

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

レビュワー確認項目

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.77%. Comparing base (ebec3e1) to head (2bcc2cd).

Additional details and impacted files

@@           Coverage Diff           @@
##              4.3    #6135   +/-   ##
=======================================
  Coverage   82.77%   82.77%           
=======================================
  Files         480      480           
  Lines       26062    26062           
=======================================
  Hits        21573    21573           
  Misses       4489     4489           

Flag	Coverage Δ
E2E	82.77% <ø> (ø)
Unit	82.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ji-eunsoo]

@tatsuishi
ありがとうございます。動作確認して、マージします。

[shinya]

ありがとうございます。マージします。