EC-CUBE / ec-cube

EC-CUBE is the most popular e-commerce solution in Japan
https://www.ec-cube.net
Other
745 stars 648 forks source link

アップデートでcustomize.js/customize.cssが上書きされてしまう #4408

Open okazy opened 4 years ago

okazy commented 4 years ago

概要(Overview)

JS/CSSをカスタマイズできる領域としてJavaScript/CSS管理機能がある。 この機能では以下のファイルに変更を保存するが、リポジトリで管理されており、パッケージにも下記ファイルが含まれる。

html/user_data/assets/js/customize.js html/user_data/assets/css/customize.css

アップデート手順では html 配下をコピーして上書きする手順のため、カスタマイズ内容が失われてしまう。

期待する内容(Expect) or 要望 (Requirement)

該当ファイルをリポジトリの管理対象から除外し、初回編集時にファイルが自動生成されるように変更する。

再現手順(Procedure)

該当ファイルを編集した上でアップデートプラグインで本体をアップデートすると該当ファイルが上書きされる

環境 (environment)

関連情報 (Ref)

https://github.com/EC-CUBE/ec-cube/pull/4083 https://github.com/EC-CUBE/ec-cube/pull/4084

nanasess commented 4 years ago

パッケージングする際に削除でも良い気がします

chihiro-adachi commented 3 years ago

管理画面側は、ファイルが存在しない場合は新規作成の挙動になります。 フロントから参照する際に404になるため、フロント側でファイルの存在チェックが必要そうです。