search

chamilo / chash

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

chash files:clean_data_files deletes courses #17

Closed ywarnier closed 10 years ago

ywarnier commented 10 years ago

In the clean_data_files action, the finder is getDataFolders(), which returns the contents of the "courses/" directory.

This is confusing (actually, the name of the command is confusing) as the comment at the beginning of the class shows "Clean the archives directory, leaving only index.html, twig and Serializer".

I think the command should be renamed to CleanTempFiles

ywarnier commented 10 years ago

Sorry, I just saw there is already a CleanTempFolder command, so the CleanDataFiles command should be renamed CleanCoursesFiles

ywarnier commented 10 years ago

OK, the cleantempfolder also deletes the archive directory itself! Passing to "bug"