The prestashop dir must be declared as a volume about docker

[Quetzacoalt91]

I'm also facing the issue about volumes today, with busy resources. I'd need to set the whole /var/www/html as a volume.

I'll have a look on your PR.

[Quetzacoalt91]

Hello @khena,

With the PR #87 updating all the PrestaShop images, I close this issue as it is almost available on Docker Hub.

Regards

[thomas15v]

I think only /var/www/html/config should be declared as volume, but only if it doesn't contain files (which I ain't sure atm). Declaring /var/www/html as volume will delete all the prestashop files, I don't think that would be a good idea.

[Quetzacoalt91]

This could work, I reapply all the files until the shop is installed. See https://github.com/PrestaShop/docker/blob/master/config_files/docker_run.sh#L11

[khena]

@thomas15v if I understand how prestashop works (I'm only a docker expert :p ) you never update the core of the framework, you will always update the 3 volumes (module, override and theme).

But we can't protect the config, admin & install dir because pretashop needs to rename these directories after install.

So I think the only solution is to mount /var/www/html and to detect on docker CMD if prestashop is already installed into the /var/www/html (for example a file exists). If yes, we skip all. If the user restart the container, we will keep the config, adminXXX and the installXXX .

it's the same kind of script than the @Quetzacoalt91 script (/tmp/ps-extractor.sh)

[khena]

PR #83

what do you think about adding a new ENV var PS_ERASE_MYSQL (default 0) only to confirm the mysql erase action?