Doctrine\Common - Githubissues

EC-CUBE / next-poc

Other

2 stars 4 forks source link

概要(Overview・Refs Issue)

Collection/ArrayCollectionについて

In ArrayCollection.php line 61: [TypeError] Doctrine\Common\Collections\ArrayCollection::__construct(): Argument #1 ($e lements) must be of type array, Eccube\ORM\Collections\ArrayCollection give n, called in /home/runner/work/next-poc/next-poc/vendor/doctrine/orm/lib/Do ctrine/ORM/UnitOfWork.php on line 683

→ Eccube\ORM\Collections\ArrayCollectionもCollectionを実装する方針で対応する

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

[ ] 既存機能の仕様変更はありません

[ ] フックポイントの呼び出しタイミングの変更はありません

[ ] フックポイントのパラメータの削除・データ型の変更はありません

[ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません

[ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません

[ ] 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

[ ] 動作確認

[ ] コードレビュー

[ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)

[ ] 互換性が保持されているか

[ ] セキュリティ上の問題がないか

[ ] 権限を超えた操作が可能にならないか
[ ] 不要なファイルアップロードがないか
[ ] 外部へ公開されるファイルや機能の追加ではないか
[ ] テンプレートでのエスケープ漏れがないか

Codecov Report

Merging #49 (0333f5f) into main (60179f7) will decrease coverage by 0.03%. The diff coverage is 76.92%.

:exclamation: Current head 0333f5f differs from pull request most recent head 0685f31. Consider uploading reports for the commit 0685f31 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@             Coverage Diff              @@
##               main      #49      +/-   ##
============================================
- Coverage     79.60%   79.58%   -0.03%     
- Complexity     7052     7088      +36     
============================================
  Files           565      567       +2     
  Lines         27798    27866      +68     
============================================
+ Hits          22129    22177      +48     
- Misses         5669     5689      +20

Flag	Coverage Δ
E2E	`68.95% <66.15%> (-0.04%)`	:arrow_down:
Unit	`77.17% <76.92%> (-0.02%)`	:arrow_down:

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

Impacted Files	Coverage Δ
...n/Eccube/Controller/Admin/Order/EditController.php	`84.14% <ø> (ø)`
...cube/Controller/Admin/Order/ShippingController.php	`89.56% <ø> (ø)`
...ntroller/Admin/Setting/Shop/DeliveryController.php	`85.92% <ø> (ø)`
...c/application/Eccube/Form/Type/Admin/OrderType.php	`95.73% <ø> (ø)`
...pplication/Eccube/Form/Type/Shopping/OrderType.php	`98.43% <ø> (ø)`
...pplication/Eccube/Repository/ProductRepository.php	`97.44% <ø> (ø)`
...rc/application/Eccube/Service/CsvExportService.php	`85.07% <ø> (ø)`
src/application/Eccube/Service/OrderHelper.php	`97.76% <ø> (ø)`
src/application/Eccube/Service/PluginService.php	`58.68% <0.00%> (-0.32%)`	:arrow_down:
...ervice/PurchaseFlow/Processor/PaymentValidator.php	`94.87% <ø> (ø)`
... and 19 more

EC-CUBE / next-poc

Doctrine\Common #49

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

レビュワー確認項目

Codecov Report