ec-cube
ec-cube copied to clipboard
詳細検索での電話番号のバリデーションを修正
概要(Overview・Refs Issue)
会員検索のバリデーションのメッセージが一貫していない close https://github.com/EC-CUBE/ec-cube/issues/6170
方針(Policy)
実装に関する補足(Appendix)
電話番号は数字のみが登録可能な仕様になっていたため、検索フォームも同様に数字のみで検索できるようにバリデーションを修正しました。
テスト(Test)
以下2つの画面が対象となります。
- 管理画面 > 会員管理 > 会員一覧
- 管理画面 > 受注管理 > 受注一覧
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
- [x] 既存機能の仕様変更はありません
- [x] フックポイントの呼び出しタイミングの変更はありません
- [x] フックポイントのパラメータの削除・データ型の変更はありません
- [x] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [x] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [x] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.75%. Comparing base (
320adb9
) to head (feebdad
).
Additional details and impacted files
@@ Coverage Diff @@
## 4.3 #6189 +/- ##
==========================================
- Coverage 82.76% 82.75% -0.01%
==========================================
Files 480 480
Lines 26431 26419 -12
==========================================
- Hits 21876 21864 -12
Misses 4555 4555
Flag | Coverage Δ | |
---|---|---|
E2E | 82.75% <100.00%> (-0.01%) |
:arrow_down: |
Unit | 82.75% <100.00%> (-0.01%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@mmikri PRありがとうございます。 現状ハイフンが入力された状態で、検索ができる仕様になっております。 この仕様は維持されますでしょうか?
@dotani1111 ありがとうございます。仕様は維持されております。 ハイフンが入力された状態で、検索をかけたとしても、エラーは発生せず、検索できることを確認しました。
動作確認してマージします。