Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

プラグインのインストールに合わせてEntity拡張機能によるスキーマ更新を行う対応 #2531

Merged
merged 11 commits into from
Oct 4, 2017

Conversation

kiy0taka
Copy link
Contributor

@kiy0taka kiy0taka commented Oct 2, 2017

概要(Overview・Refs Issue)

実装に関する補足(Appendix)

  • テスト時にスキーマ変更を行うと後続テストが失敗してしまうため、プロキシの再生成とスキーマ変更をそれぞれEntityProxyServiceSchemaServiceに分割してテスト時にはMockオブジェクトを差し込めるようにしています。

テスト(Test)

  • プロキシ再生成のテストは追加しましたが、スキーマ変更のテストは他のテストに影響がでるため自動テストはありません。

相談(Discussion)

  • 同じプロセス内でロードされたEntityクラスに対応するプロキシを生成し、そのプロキシからアノテーションを読み込むために、ReloadSafeAnnotationDriverの実装が少し強引になっています。
  • プラグインの有効化/無効化でスキーマ変更が自動的に行われるので、今後はPluginManagerからのスキーマ変更処理実行は不要になると思われる。

@kiy0taka kiy0taka added this to the 3.1.0 milestone Oct 2, 2017
@chihiro-adachi chihiro-adachi merged commit f1298ff into EC-CUBE:experimental/3.1 Oct 4, 2017
@kiy0taka kiy0taka deleted the dev/proxy-gen branch October 4, 2017 06:29
@Yangsin Yangsin added the enhancement 機能追加 label Oct 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants