Projet

Général

Profil

Bug #32469

mauvais chemin par défaut lors de l'import des templates

Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
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

test_import_template.py (1,77 ko) test_import_template.py test Nicolas Roche (absent jusqu'au 3 avril), 18 avril 2019 18:03
0001-add-tests-on-import-template-query-32469.patch (3,3 ko) 0001-add-tests-on-import-template-query-32469.patch Nicolas Roche (absent jusqu'au 3 avril), 18 avril 2019 18:51
0001-add-tests-on-import-template-query-32469.patch (4,51 ko) 0001-add-tests-on-import-template-query-32469.patch Nicolas Roche (absent jusqu'au 3 avril), 19 avril 2019 10:10

Demandes liées

Lié à Hobo - Development #14630: Avoir un import-template génériqueFermé17 janvier 2017

Actions

Révisions associées

Révision 6eb6a6c5 (diff)
Ajouté par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

add tests on import-template query (#32469)

Historique

#1

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

#2

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

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.

#3

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)
#4

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.

#5

Mis à jour par Nicolas Roche (absent jusqu'au 3 avril) il y a presque 5 ans

fait :
- déplacement du test du fichier dans le bloc if
- retour au warning

#6

Mis à jour par Christophe Siraut il y a presque 5 ans

  • Statut changé de Solution proposée à Solution validée
#7

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)
#8

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

Formats disponibles : Atom PDF