Entityのproxy生成時にifを削除しないよう変更

[izayoi256]

概要(Overview・Refs Issue)

#4718 の参考実装。

#4708 によって既存の本体Entityの拡張には追従可能だが、カスタマイズEntityやプラグインEntityでclass_exists(Entity::class)を利用している場合に支障をきたすか。

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

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

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[okazy]

なぜかpgsqlのテストだけ結構落ちていたのでいったん再実行してみました。
ざっとみたところ落ちている箇所はproxy関連でした。

[izayoi256]

確認して修正してみます。

[izayoi256]

___ *** ERROR *** ==> ADMIN ________________________________________________________________________
システムエラーが発生しました。
--> - 'Class ''Eccube\Entity\Cart2E6CEulBdyah'' does not exist'
--> - /home/runner/work/ec-cube/ec-cube/src/Eccube/Service/PluginService.php
--> - 283
--> - "#0 /home/runner/work/ec-cube/ec-cube/src/Eccube/Service/PluginService.php(238): Eccube\Service\PluginService->postInstall(Array, 100)\n#1 /home/runner/work/ec-cube/ec-cube/src/Eccube/Controller/Admin/Store/PluginController.php(330): Eccube\Service\PluginService->installWithCode('Horizon')\n#2 /home/runner/work/ec-cube/ec-cube/vendor/symfony/http-kernel/HttpKernel.php(151): Eccube\Controller\Admin\Store\PluginController->enable(Object(Eccube\Entity\Plugin), Object(Eccube\Util\CacheUtil), Object(Symfony\Component\HttpFoundation\Request))\n#3 /home/runner/work/ec-cube/ec-cube/vendor/symfony/http-kernel/HttpKernel.php(68): Symfony\Component\HttpKernel\HttpKernel->handleRaw(Object(Symfony\Component\HttpFoundation\Request), 1)\n#4 /home/runner/work/ec-cube/ec-cube/vendor/symfony/http-kernel/Kernel.php(200): Symfony\Component\HttpKernel\HttpKernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)\n#5 /home/runner/work/ec-cube/ec-cube/index.php(76): Symfony\Component\HttpKernel\Kernel->handle(Object(Symfony\Component\HttpFoundation\Request))\n#6 {main}"
--> extra: { url: /admin/store/plugin/1/enable, ip: '::1', http_method: POST, server: localhost, referrer: 'http://localhost:8000/admin/store/plugin', user_agent: 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.102 Safari/537.36', file: /home/runner/work/ec-cube/ec-cube/src/Eccube/Log/Logger.php, line: 64, class: Eccube\Log\Logger, function: log, uid: '0503741', user_id: 1, session_id: 8be99499 }

artifactsを探してみましたが恐らくこれでしょうか？

ちょっとcodeceptionの知見がなく、CIと同様の条件で試すことができず…。試しにdev/prodそれぞれでdumpautoloadした上でcodeception/_data/plugins/Horizon-1.0.0.tgzを管理画面上でインストール->有効化->無効化->削除と試してみましたが、エラーのたぐいを発生させることはできませんでした。

もしアレだったらクローズしてしまってください…。

[xuelian311]

関連issueがクローズされているため、prもクローズします。