[4.2]WebAPI プラグインの削除に失敗する問題の修正

[nanasess]

概要(Overview・Refs Issue)

WebAPI プラグインを削除する際, 以下のようなエラーが発生するのを修正

MappingException.php line 63:
File mapping drivers must have a valid directory path, however the given path [/path/to/ec-cube/vendor/league/oauth2-server-bundle/src/Entity] seems to be incorrect!

ClassMetadataFactory が既に削除された Bundle のパスを参照してしまうため MappingException が発生してしまう。
関連テーブルは ComposerApiService:dropTableToExtra() で既に削除されているため無視しても問題ないと思われる

方針(Policy)

 - generateProxyAndUpdateSchema 及び dropTable を try/catch で囲む

実装に関する補足(Appendix)

MappingDriver を何とかしようとしたが、ClassMetadataFactory のメソッドをコールしただけで MappingException が発生してしまう。

テスト（Test)

• WebAPI プラグインが正常に削除できるのを確認
• plugin-test の GitHub Actions が正常に通るのを確認

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5418 (c739e5b) into 4.2 (9f37496) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                4.2    #5418      +/-   ##
============================================
- Coverage     53.04%   53.01%   -0.04%     
- Complexity     6155     6157       +2     
============================================
  Files           466      466              
  Lines         20932    20935       +3     
============================================
- Hits          11104    11098       -6     
- Misses         9828     9837       +9     

Flag	Coverage Δ
E2E	49.46% <0.00%> (-0.04%)	⬇️
Unit	11.34% <0.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
src/Eccube/Service/PluginService.php	12.27% <0.00%> (-0.10%)	⬇️
...orage/Handler/SameSiteNoneCompatSessionHandler.php	76.11% <0.00%> (-2.99%)	⬇️
.../Eccube/Form/Type/Admin/ProductClassMatrixType.php	86.11% <0.00%> (-2.78%)	⬇️
...rc/Eccube/Form/Type/Admin/ProductClassEditType.php	87.01% <0.00%> (-1.30%)	⬇️
...ontroller/Admin/Product/ProductClassController.php	80.57% <0.00%> (-1.15%)	⬇️

---

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 9f37496...c739e5b. Read the comment docs.

[chihiro-adachi]

@nanasess
ありがとうございます、修正確認しました。