PHP8とMySQLでプラグインアップデートコマンドを実行すると no active transaction になるのを修正

[kurozumi]

概要(Overview・Refs Issue)

PHP8とMySQLでプラグインアップデートコマンドを実行すると no active transaction になるのを修正しました。

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

プラグインテストのワークフローにPHP8.0と8.1を追加しました。

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5443 (11db57d) into 4.2 (0b3fde8) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                4.2    #5443      +/-   ##
============================================
- Coverage     77.49%   77.48%   -0.01%     
- Complexity     6234     6250      +16     
============================================
  Files           466      468       +2     
  Lines         20962    21007      +45     
============================================
+ Hits          16244    16277      +33     
- Misses         4718     4730      +12     

Flag	Coverage Δ
E2E	49.49% <0.00%> (+<0.01%)	⬆️
Unit	76.99% <0.00%> (+0.02%)	⬆️

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

Impacted Files	Coverage Δ
src/Eccube/Service/PluginService.php	55.58% <0.00%> (-0.43%)	⬇️
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	75.43% <0.00%> (-3.51%)	⬇️
src/Eccube/Service/CsvImportService.php	71.96% <0.00%> (-1.96%)	⬇️
.../Eccube/Stream/Filter/SjisToUtf8EncodingFilter.php	80.64% <0.00%> (ø)
src/Eccube/Stream/Filter/ConvertLineFeedFilter.php	100.00% <0.00%> (ø)
...e/Controller/Admin/Product/CsvImportController.php	64.66% <0.00%> (+0.04%)	⬆️
.../Eccube/Doctrine/Common/CsvDataFixtures/Loader.php	80.00% <0.00%> (+5.00%)	⬆️
...e/Controller/Admin/AbstractCsvImportController.php	92.30% <0.00%> (+7.30%)	⬆️

---

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 0b3fde8...11db57d. Read the comment docs.

[chihiro-adachi]

@kurozumi
mysqlでテストが落ちているようなのでご確認いただけますか？
おそらくDATABASE_CHARSETを指定してあげればいけるかと。

[chihiro-adachi]

@kurozumi
手元の環境で試してみてみたところ、TransactionListener::onKernelTerminateでも同じエラーが発生してました。

TransactionListenerではisTransactionActiveでトランザクション中の判定をしてますが、これがうまくできないようで、PluginServiceの実装にあわせるとテストも通るようです。

chihiro-adachi@efa0558

[kurozumi]

@chihiro-adachi

修正箇所はPluginServiceのpostInstallの実装に合わせました。

https://github.com/kurozumi/ec-cube/blob/11db57d34a75e33a739296d29236324446f5a47d/src/Eccube/Service/PluginService.php#L782-L790

[chihiro-adachi]

@kurozumi

修正箇所はPluginServiceのpostInstallの実装に合わせました。
はい、そちらはその修正で問題ありません。

TransactionListener::onKernelTerminateでも同じエラーが発生しており、テストが落ちているので、TransactionListenerも合わせて修正いただければと。

以下の箇所です。
https://github.com/EC-CUBE/ec-cube/blob/4.2/src/Eccube/EventListener/TransactionListener.php#L126

[chihiro-adachi]

@kurozumi
ご対応ありがとうございました。
確認後マージさせていただきます。