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

[4.1] eccube:install コマンドでプラグインの有効処理を追加 #4787

Merged
merged 3 commits into from
Dec 7, 2020

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Dec 3, 2020

概要(Overview・Refs Issue)

方針(Policy)

  • APIプラグインはデフォルトで有効化する

実装に関する補足(Appendix)

  • 現状、プラグインコードをハードコーディングしている
  • APIプラグインが存在しない場合、エラーメッセージを表示し、インストールは正常終了する

テスト(Test)

相談(Discussion)

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

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

レビュワー確認項目

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

@nanasess nanasess force-pushed the installer-command-plugin-enable branch from 4096e95 to eff1fb9 Compare December 3, 2020 07:11
@okazy okazy added this to the 4.1 milestone Dec 4, 2020
Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認しました。

WebAPIプラグインがインストールされていない状態(git clone & composer install)

問題なくEC-CUBEのインストールができました。

WebAPIプラグインがインストールされている状態(#4784 のパッケージ)

EC-CUBEのインストールでWebAPIプラグインのインストールまでできました。
ただ、キャッシュクリアがうまくできていないのか、インストール直後にアクセスするとプラグインが有効化されていない状態で、その後キャッシュをクリアするとプラグインが有効になりました。
eccube:plugin:enable コマンド内でキャッシュクリアの処理はあり、インストール時のログからキャッシュクリアは走っているようです。
eccube:plugin:enable を個別に実行した場合も同様の問題が発生したので、 eccube:plugin:enable コマンド自体の問題かと思います。

本プルリクの問題ではありませんが直しておきたい・・・

@okazy
Copy link
Contributor

okazy commented Dec 4, 2020

キャッシュの削除処理を変えてみました。
nanasess#40

コマンドのキャッシュクリア処理を別プロセスで実行
@nanasess
Copy link
Contributor Author

nanasess commented Dec 4, 2020

@okazy ありがとうございます。マージしました。

@okazy
Copy link
Contributor

okazy commented Dec 7, 2020

ありがとうございます!

@okazy okazy merged commit ed3b975 into EC-CUBE:4.1 Dec 7, 2020
@okazy okazy changed the title eccube:install コマンドでプラグインの有効処理を追加 [4.1] eccube:install コマンドでプラグインの有効処理を追加 Dec 7, 2020
@nanasess nanasess deleted the installer-command-plugin-enable branch October 3, 2022 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants