ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

詳細検索での電話番号のバリデーションを修正

Open mmikri opened this issue 9 months ago • 4 comments

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

mmikri avatar May 08 '24 14:05 mmikri

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.

codecov[bot] avatar May 09 '24 00:05 codecov[bot]

@mmikri PRありがとうございます。 現状ハイフンが入力された状態で、検索ができる仕様になっております。 この仕様は維持されますでしょうか?

dotani1111 avatar May 13 '24 05:05 dotani1111

@dotani1111 ありがとうございます。仕様は維持されております。 ハイフンが入力された状態で、検索をかけたとしても、エラーは発生せず、検索できることを確認しました。

mmikri avatar May 13 '24 11:05 mmikri

動作確認してマージします。

ji-eunsoo avatar May 15 '24 07:05 ji-eunsoo