ec-cube
ec-cube copied to clipboard
検索結果が数値の場合は、カンマを表示するように修正
概要(Overview・Refs Issue)
#5751のissueに対応しました
方針(Policy)
数値にはカンマをつける、としたため
購入件数にもカンマが入るようになっています
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
- [ ] 既存機能の仕様変更はありません
- [ ] フックポイントの呼び出しタイミングの変更はありません
- [ ] フックポイントのパラメータの削除・データ型の変更はありません
- [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
Codecov Report
Merging #5752 (386ca05) into 4.2 (11e7175) will decrease coverage by
0.00%
. The diff coverage is100.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.
@takeuji テストが落ちているようなのでご確認いただけますでしょうか。
@takeuji ご対応ありがとうございます。 修正問題ないかと思います。