Add a CLI to remove temporary files #66
Conversation
|
|
||
| foreach ($this->storage->all() as $filePath) { | ||
| $this->storage->remove($filePath); | ||
| } |
There was a problem hiding this comment.
attention il ne faut pas tout supprimer, seuls les vieux fichiers doivent etre supprimés
d'autre part, il faut aussi suppirmer les entités liés aux fichiers
pour moi, il faut un paramètre optionnel permettant de définir l'age des fichiers à supprimer, et supprimer uniquement ceux la. Si option vide, on supprime tout.
idée pour le format de l'option : --older-than=3d
ping @Vowow
There was a problem hiding this comment.
@juchi Ok aussi ça permet d'être plus précis si besoin
There was a problem hiding this comment.
Ok pour le paramètre older-than, c'est corrigé :)
On est d'accord que dans ce cas l'objectif est de supprimer les fichiers temporaires (qui ne sont donc plus utilisés) ? Pour moi ils ne sont liés à aucune entité, puisque les fichiers réellement uploadés sont copiés dans un autre répertoire au moment de la création de l'entité.
NeilPeyssard
force-pushed
the
feature/remove_tmp_files_cli
branch
from
1e94aa7
to
c61836b
Compare
No description provided.