電話番号、郵便番号バリデーションで不適切な文字が通過するので修正

[kazuyoshikakihara]

概要(Overview・Refs Issue)

郵便番号、電話番号のバリデーションにis_numericが使われているため、番号としてはありえない「3.0e2」などの値が通過してしまいます。
これが原因で、ユーザーが間違ってピリオドなどを入力してしまったときなどに検出できないことがあります。

ctype_digitに置き換えたほうが厳密に数字だけチェックできますし、置き換えたからといって使い勝手が悪くなることもありません。

テスト（Test)

自動テスト用のテストコードを書きました。

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5482 (d988869) into 4.2 (9bafbe4) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                4.2    #5482      +/-   ##
============================================
+ Coverage     78.74%   78.77%   +0.02%     
  Complexity     6251     6251              
============================================
  Files           468      468              
  Lines         21007    21010       +3     
============================================
+ Hits          16543    16551       +8     
+ Misses         4464     4459       -5     

Flag	Coverage Δ
E2E	64.35% <100.00%> (+0.01%)	⬆️
Unit	77.49% <100.00%> (+0.50%)	⬆️

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

Impacted Files	Coverage Δ
src/Eccube/Form/Type/PhoneNumberType.php	95.45% <ø> (ø)
src/Eccube/Form/Type/PostalType.php	95.45% <ø> (ø)
.../Eccube/Controller/NonMemberShoppingController.php	92.08% <100.00%> (+1.43%)	⬆️
src/Eccube/Form/Type/Install/Step3Type.php	100.00% <0.00%> (ø)
src/Eccube/Form/Type/Admin/ChangePasswordType.php	94.11% <0.00%> (+0.36%)	⬆️
...ontroller/Admin/Product/ProductClassController.php	90.85% <0.00%> (+0.57%)	⬆️
src/Eccube/Form/Type/RepeatedPasswordType.php	85.00% <0.00%> (+0.78%)	⬆️
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	78.94% <0.00%> (+3.50%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9bafbe4...d988869. Read the comment docs.

[chihiro-adachi]

@kazuyoshikakihara
ありがとうございます。問題ないかと思います。