商品画像削除時、他の商品画像が参照するファイルは削除しないよう変更

[izayoi256]

概要(Overview・Refs Issue)

#4752 の内、ファイル削除に関する処理の見直し。

方針(Policy)

ProductImageエンティティを削除後、同ファイル名のProductImageが存在しない場合のみファイルを削除。

テスト（Test)

ユニットテストを追加。

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

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

レビュワー確認項目

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

[okazy]

@izayoi256

プルリクありがとうございます！
動作は確認できていないのですが、商品自体の削除時も画像の削除が行われていますので、対策が必要かと思いました。

ec-cube/src/Eccube/Controller/Admin/Product/ProductController.php

Lines 721 to 729 in 9de60f8

	// 画像ファイルの削除(commit後に削除させる)
	foreach ($deleteImages as $deleteImage) {
	try {
	$fs = new Filesystem();
	$fs->remove($this->eccubeConfig['eccube_save_image_dir'].'/'.$deleteImage);
	} catch (\Exception $e) {
	// エラーが発生しても無視する
	}
	}

[izayoi256]

@okazy
ありがとうございます。そちらも併せて対応しました。

[okazy]

ソースコード問題ありません。
以下動作確認して重複登録画像が削除されないことを確認しました。

• 商品画像削除
• 商品自体の削除
• CSVのフラグでの商品削除
• CSVで画像Pathを除外（既存の仕様）

[okazy]

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