Bug #15247
trace lors d'une sauvegarde d'une annonce contenant des images distantes
Début:
05 mars 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
OSError: [Errno 13] Permission denied: '/usr/lib/python2.7/dist-packages/media'
(10 additional frame(s) were not displayed)
...
File "corbo/forms.py", line 37, in save
instance = super(AnnounceForm, self).save(*args, **kwargs)
File "corbo/models.py", line 108, in save
dirs, files = storage.listdir(self.images_path)
File "corbo/models.py", line 144, in images_path
os.makedirs(os.path.join(storage.location, path))
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a environ 7 ans
- Fichier 0001-fix-announce-images-storage-path-15247.patch 0001-fix-announce-images-storage-path-15247.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mauvais calcul du chemin vers le "media" de l'instance.
Mis à jour par Frédéric Péters il y a environ 7 ans
assert os.path.exists(storage.path(os.path.join('images', str(announce.id), '01_logo.png')))
Ça passe même sans la modif à corbo/models.py.
Mis à jour par Serghei Mihai il y a environ 7 ans
Il faudrait que j'arrive à mock-er la méthode path du storage, mais ce n'est pas évident car c'est un objet lazy qui depend si on est en mode multitenant ou pas.
J'essaie de trouver une solution.
Mis à jour par Frédéric Péters il y a environ 7 ans
Si c'est trop compliqué on peut zapper ça, par contre ce serait quand même bien que MEDIA_ROOT soit posé différemment pour les tests, parce que là ça écrit dans les sources.
Mis à jour par Serghei Mihai il y a environ 7 ans
- Fichier 0001-fix-announce-images-storage-path-15247.patch 0001-fix-announce-images-storage-path-15247.patch ajouté
Yep, dans /tmp/media
Mis à jour par Frédéric Péters il y a environ 7 ans
Yep, dans /tmp/media
Il faut un endroit temporaire, pour ne pas qu'une exécution ultérieure qui serait buggée passe parce qu'il restait des traces d'une précédente exécution.
Mis à jour par Serghei Mihai il y a environ 7 ans
Mis à jour par Serghei Mihai il y a environ 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit b0428f98a8ddc21e39832cdad2a5178b65f778d2
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Sun Mar 5 23:37:42 2017 +0100
fix announce images storage path (#15247)
fix announce images storage path (#15247)