不必要に二重に呼び出されているコンタクトフォームの処理を削除

[eternalharvest]

概要(Overview・Refs Issue)

お問い合わせ画面で入力した内容を処理し、確認画面をレンダリングする際に二重に $form->handleRequest() が呼び出されていたため、プラグイン開発時に FormExtension でイベント処理をすると二重に発火されて困っていました。一度しか実行されてほしくないため、またそもそも２回も呼び出す必要がないため削除しました。

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[okazy]

確かに2回目の handleRequest は不要かと思います。
動作確認をして問題なく問い合わせメールが送信されることを確認しました。

[okazy]

ありがとうございます！取り込みました。