fix: #4824 PDFのロゴ画像を空画像に変更

[tao-s]

概要(Overview・Refs Issue)

#4824 PDFのロゴ画像を空画像に変更。これでとりあえずそのまま使える。ロゴ画像入れたい人だけ後で変える方向で。

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

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

レビュワー確認項目

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

[chihiro-adachi]

@tao-s
ありがとうございます。
動作確認できましたらマージします。

[okazy]

キャッシュが残っているのか、手元の環境でロゴの変更ができませんでした。
考えられる原因があれば教えてください。

変更前

変更後

[tao-s]

@okazy キャッシュクリアしてもなりますか？

[tao-s]

@okazy ユーザ領域にロゴ画像生成されていませんかね？

ec-cube/src/Eccube/Service/OrderPdfService.php

Line 320 in 3b4bfd4

// user_dataにlogo.pngが配置されている場合は優先的に読み込む

[okazy]

@tao-s

html/user_data/assets/pdf/logo.png を削除すればいけました！

html/user_data/assets/pdf/logo.png も白い画像に差し替えた方が良いかと思いましたがいかがでしょうか？

[tao-s]

@okazy html/user_data/assets/pdf/logo.pngって最初からファイルありましたっけ？インストール時かなんかの時に自動でできた気がするんですが

[okazy]

@tao-s

ありがとうございます！
確かにインストール時に展開処理が入っていました。
インストール後のブランチを切り替えてテストをしてたためうまくいっていませんでした。

ec-cube/src/Eccube/Controller/Install/InstallController.php

Lines 233 to 240 in cb0b554

	$logoPath = '/assets/pdf/logo.png';
	if (!file_exists($this->getParameter('eccube_html_dir').'/user_data'.$logoPath)) {
	$file = new Filesystem();
	$file->copy(
	$this->getParameter('eccube_html_admin_dir').$logoPath,
	$this->getParameter('eccube_html_dir').'/user_data'.$logoPath
	);
	}

ec-cube/src/Eccube/Command/LoadDataFixturesEccubeCommand.php

Lines 128 to 135 in cb0b554

	$logoPath = '/assets/pdf/logo.png';
	if (!file_exists($this->container->getParameter('eccube_html_dir').'/user_data'.$logoPath)) {
	$file = new Filesystem();
	$file->copy(
	$this->container->getParameter('eccube_html_admin_dir').$logoPath,
	$this->container->getParameter('eccube_html_dir').'/user_data'.$logoPath
	);
	}

[okazy]

テストして問題ありませんでした。

[okazy]

ありがとうございます！取り込みました。