Dolibarr / dolibarr-docker

Official Dolibarr docker images. Based on the work of Garcia MICHEL (tuxgasy)
https://hub.docker.com/r/dolibarr/dolibarr
GNU General Public License v3.0
16 stars 3 forks source link

v20 PHP-tidy not enabled by default #16

Open JonBendtsen opened 2 days ago

JonBendtsen commented 2 days ago

MAIN_RESTRICTHTML_ONLY_VALID_HTML_TIDY = Undefined (Recommended: 1) - Module "php-tidy" must be enabled (currently: Not available)

V20 dolibarr image.

mathieupotier commented 7 hours ago

@JonBendtsen which module are you refering too ? I don't have this on my instance ...

@creekorful what about having it loaded by default or having an extra ENV_VAR at runtime for enabling it on container run... I think this image should ship with the minimal loaded by default (to optimise the runtime) and let people to enable php_module (if available) by setting an env_var... I think about this operation in docker-run init script ... https://github.com/Dolibarr/dolibarr-docker/blob/4facf6427c02571f94771859fa52126f9cf259bd/docker-run.sh#L45

JonBendtsen commented 6 hours ago

@JonBendtsen which module are you refering too ? I don't have this on my instance ...

yeah, sorry about not being clear. I think that this is once again:

Home, Admin Tools, About Security page, About PHP or About performance pages

JonBendtsen commented 1 hour ago

this is from /admin/system/security.php?mainmenu=home&leftmenu=admintools

Home, Admin Tools, About Security