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

Copy favicon to user_data directory #4075

Merged
merged 16 commits into from Feb 13, 2019

Conversation

@nanasess
Copy link
Contributor

commented Feb 7, 2019

概要(Overview・Refs Issue)

  • #4070
  • favicon を user_data 以下へ移動する

方針(Policy)

  • Symfony Assets の package に user_data を追加
  • インストーラで favicon を user_data/assets 以下へコピーする
  • default_frame.twig の favicon の asset に user_data を追加
  • faviconは、html/user_data/assets/img/common/favicon.icoを参照する

テスト(Test)

  • テストケースを追加

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

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

4.0.1 からの仕様変更内容

src/Eccube/Resource/template/default/default_frame.twig を直接カスタマイズしている方

以下の修正を適用することで, ファイル管理機能から favicon をアップロードできるようになります
50fcbde#diff-7cefac9fd3759d999afb711a36b6dad9

app/template/%eccube.theme%/default_frame.twig デザインテンプレートを設定している方

app/template/%eccube.theme%/default_frame.twig に以下の修正を適用することで, ファイル管理機能から favicon をアップロードできるようになります
50fcbde#diff-7cefac9fd3759d999afb711a36b6dad9

app/template/default_frame.twig を設置している方

app/template/default_frame.twig に以下の修正を適用することで, ファイル管理機能から favicon をアップロードできるようになります
50fcbde#diff-7cefac9fd3759d999afb711a36b6dad9

デザインテンプレート制作者の方

default_frame.twig に以下の修正を適用することで, ファイル管理機能から favicon をアップロードできるようになります
50fcbde#diff-7cefac9fd3759d999afb711a36b6dad9

html/template/default/assets/img/common/favicon.ico を変更している方

default_frame.twig の変更を適用しないようお願いいたします
ファイル管理機能から favicon をアップロードする機能はご利用いただけません。

html/template/%eccube.theme%/assets/img/common/favicon.ico を変更している方

default_frame.twig の変更を適用しないようお願いいたします
ファイル管理機能から favicon をアップロードする機能はご利用いただけません。

nanasess added some commits Feb 7, 2019

@nanasess

This comment has been minimized.

Copy link
Contributor Author

commented Feb 7, 2019

AppVeyor が落ちていますが #4074 が取り込まれれば解決すると思われます

@t-nagahashi t-nagahashi requested review from okazy and chihiro-adachi Feb 7, 2019

yuyu777 added some commits Feb 8, 2019

Merge pull request #4075 from nanasess/move-favicon
Copy favicon to user_data directory
@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 8, 2019

@nanasess
scrutinizerの修正をおねがいします

@yuyu777 yuyu777 referenced this pull request Feb 8, 2019
6 of 6 tasks complete

@nanasess nanasess closed this Feb 12, 2019

@nanasess nanasess reopened this Feb 12, 2019

@nanasess

This comment has been minimized.

Copy link
Contributor Author

commented Feb 12, 2019

@chihiro-adachi 修正してみましたが、、現在、 composer.json に記載されている 1.9.x ではなく、 1.10.1 をロードしてますね。。
1.9.x では deprecated になっていないようです
https://github.com/doctrine/DoctrineBundle/blob/1.9.x/Command/DoctrineCommand.php

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 12, 2019

修正してみましたが、、現在、 composer.json に記載されている 1.9.x ではなく、 1.10.1 をロードしてますね。。

対応ありがとうございます。
#4074 で上がったみたいですね。

@chihiro-adachi

This comment has been minimized.

Copy link
Contributor

commented Feb 12, 2019

@nanasess
src/Eccube/Resource/template/default/error.twigのほうも修正おねがいできますでしょうか。
https://github.com/EC-CUBE/ec-cube/blob/4.0/src/Eccube/Resource/template/default/error.twig#L17

@nanasess

This comment has been minimized.

Copy link
Contributor Author

commented Feb 12, 2019

@chihiro-adachi 修正しました

nanasess and others added some commits Feb 12, 2019

Merge pull request #4080 from yuyu777/dispPath-atFileManager
FileManager:display filePath when you upload a file.

@chihiro-adachi chihiro-adachi merged commit b0cf11d into EC-CUBE:4.0 Feb 13, 2019

3 checks passed

Inspection Summary
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.