Bug #32469
mauvais chemin par défaut lors de l'import des templates
Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
18 avril 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Le chemin pour importer les templates ne semble pas correct.
$ chrono-manage import_template please_import_me --no-color ... Unknown template ('None/please_import_me.json')
Cf le test joint.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans
- Lié à Development #14630: Avoir un import-template générique ajouté
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans
- Fichier 0001-add-tests-on-import-template-query-32469.patch 0001-add-tests-on-import-template-query-32469.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce patch ajoute les tests sur hobo/agent/common/management/commands/import_template.py
Afin de détecter les erreurs sur le chemin du fichier template, j'ai remplacé le warning par une exception.
Mis à jour par Christophe Siraut il y a presque 5 ans
Je pense qu'on devrait déplacer la vérification du gabarit dans le bloc suivant :
if 'import_site' in get_commands(): if not os.path.exists(template): raise CommandError('Unknown template (%r)' % template) call_command('import_site', template)
Mis à jour par Frédéric Péters il y a presque 5 ans
Non, on a parfois des valeurs dans template_name qui ne correspondent pas à des fichiers réels, on ne peut pas sortir en erreur sur la situation, cf #31958.
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans
- Fichier 0001-add-tests-on-import-template-query-32469.patch 0001-add-tests-on-import-template-query-32469.patch ajouté
fait :
- déplacement du test du fichier dans le bloc if
- retour au warning
Mis à jour par Christophe Siraut il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6eb6a6c525adad8031bbae9d06b1eec9b1f5d6ed Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Thu Apr 18 18:41:19 2019 +0200 add tests on import-template query (#32469)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add tests on import-template query (#32469)