ユーザー独自プラグインのアップデートしてもスキーマがアップデートされない

[nobuhiko]

概要(Overview・Refs Issue)

• ユーザー独自プラグインのアップデートした時に、スキーマを更新してほしい

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

レビュワー確認項目

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

[kazumiiiiiiiiiii]

E2E テストで使用されているプラグインの詳細を確認予定。

[kazumiiiiiiiiiii]

関連 #4065

[junpeko5]

@nobuhiko @nanasess

動作検証したところ、
独自プラグインのアップデート時にスキーマがアップデートされませんでした。
$this->generateProxyAndUpdateSchema($plugin, $meta);
を呼び出す前に、
$em->flush();
が必要なのではと考えています。

$em->persist($plugin);
$em->flush();
$this->generateProxyAndUpdateSchema($plugin, $meta);

（今回の修正で、bin/console eccube:plugin:updateコマンドは正しく動作するようになっています。）

[okazy]

@nobuhiko さん
@junpeko5 さんからコメントいただいている点についていかがでしょうか？

E2E テストが追加できたらいいのですが、proxyの追加削除のテストの基盤がないので、テストの追加可能かどうかこちらでも確認してみます。

[nobuhiko]

@okazy
この修正で、大丈夫なことを確認していますが、どうやったら証明出来ますか？

[kiy0taka]

@nobuhiko

APP_DEBUG=1 の場合ならこの修正で動作しましたが、 APP_DEBUG=0 のときにはスキーマが更新されませんでした。($em->flush(); を入れてもだめでした。)

こちらでスキーマが更新されるように修正しましたのでご確認いただけますか？
問題なければマージさせてもらいます。

[nobuhiko]

@kiy0taka 確認しましたー

[kiy0taka]

確認ありがとうございました。