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

有効化したプラグインが無効と判定されてしまうのを修正 #4047

Merged
merged 1 commit into from
Jan 20, 2019

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Dec 13, 2018

概要(Overview・Refs Issue)

#4043 の対策

PrependExtension では、Symfony Container のパラメータはまだ有効になっていないため、 ContainerInterface::resolveEnvPlaceholders() を使用して Doctrine のパラメータを取得している。
DATABASE_URL に % が含まれていると、エスケープされてしまい、後続の DB 接続に失敗してしまう

方針(Policy)

  • DATABASE_URL を環境変数から取得し直すよう修正

テスト(Test)

  • URLエンコードされたパスワードを使用して、プラグイン設定画面のアイコンが表示されることを確認

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

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

@ghost ghost added this to the 4.0.2 milestone Dec 18, 2018
@ghost ghost added the bug:High label Dec 18, 2018
Copy link
Contributor

@chihiro-adachi chihiro-adachi left a comment

Choose a reason for hiding this comment

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

記号ありのパスワードで、有効時に設定ボタンが表示されることを確認

@chihiro-adachi chihiro-adachi merged commit 31e2494 into EC-CUBE:4.0 Jan 20, 2019
@nanasess nanasess deleted the fix-prepend-connection branch October 3, 2022 07:43
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.

2 participants