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

検索結果が数値の場合は、カンマを表示するように修正

Open takeuji opened this issue 2 years ago • 2 comments

概要(Overview・Refs Issue)

#5751のissueに対応しました

方針(Policy)

数値にはカンマをつける、としたため 購入件数にもカンマが入るようになっています image

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

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

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

レビュワー確認項目

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

takeuji avatar Aug 31 '22 12:08 takeuji

Codecov Report

Merging #5752 (386ca05) into 4.2 (11e7175) will decrease coverage by 0.00%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                4.2    #5752      +/-   ##
============================================
- Coverage     78.75%   78.74%   -0.01%     
- Complexity     6288     6289       +1     
============================================
  Files           470      470              
  Lines         21098    21100       +2     
============================================
+ Hits          16615    16616       +1     
- Misses         4483     4484       +1     
Flag Coverage Δ
E2E 64.78% <100.00%> (+<0.01%) :arrow_up:
Unit 77.78% <100.00%> (-0.01%) :arrow_down:

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

Impacted Files Coverage Δ
src/Eccube/Twig/Extension/EccubeExtension.php 85.14% <100.00%> (+0.30%) :arrow_up:
...be/Service/PurchaseFlow/Processor/TaxProcessor.php 82.45% <0.00%> (-1.76%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 31 '22 22:08 codecov-commenter

@takeuji テストが落ちているようなのでご確認いただけますでしょうか。

chihiro-adachi avatar Sep 01 '22 23:09 chihiro-adachi

@takeuji ご対応ありがとうございます。 修正問題ないかと思います。

chihiro-adachi avatar Oct 12 '22 23:10 chihiro-adachi