chamilo / chash

Chamilo Shell
GNU General Public License v3.0
15 stars 10 forks source link

files:clean_temp_folder removes the archive directory #21

Closed ywarnier closed 10 years ago

ywarnier commented 10 years ago

There is apparently a mistake in the clean_temp_folder command, as it also removes the entire archive/ directory (while it should really maintain the .htaccess and index.php file there), at least in version 1.9.*

This is a bug (and a bad one) because it can prevent the portal from working

ywarnier commented 10 years ago

The notPath() filter doesn't seem to work and, as a consequence, I cannot avoid the deletion of index.html. Apparently, although I haven't used the ignoreDotFiles() modifier, .htaccess survives the operation. Leaving open for now, until we found a solution to keep index.html

aigualada commented 10 years ago

I tested this : In getTempFiles() after lines (441) $finder->in($sysPath.'archive/') and (445) $finder->in($sysPath.'data/temp/'); add this $finder->files()->notName('index.*'); Tested and works.

ywarnier commented 10 years ago

Commited in your name, thanks!