Projet

Général

Profil

Anomalie #4538

creoled : message peu clair si on inverse -t ou -s

Ajouté par Emmanuel GARETTE il y a plus de 11 ans. Mis à jour il y a environ 11 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Daniel Dehennin
Catégorie:
-
Version cible:
Distribution EOLE - Eole 2.4-dev-2
Début:
03/12/2012
Echéance:
% réalisé:

100%

Temps passé:
3.15 h
Distribution:
EOLE 2.4

Description

root@eoleng:~# CreoleCat -d -t pouet -o pouet.tmpl
Starting new HTTP connection (1): 127.0.0.1
requests.packages.urllib3.connectionpool - "GET /get/creole HTTP/1.1" 200 3825
Starting new HTTP connection (1): 127.0.0.1
requests.packages.urllib3.connectionpool - "GET /get/containers/files HTTP/1.1" 200 15158
root - Traceback (most recent call last):
  File "/usr/bin/CreoleCat", line 101, in main
    engine._copy_to_template_dir(filevar)
  File "/usr/lib/python2.7/dist-packages/creole/template.py", line 224, in _copy_to_template_dir
    tmpl_source_name = os.path.split(filevar['source'])[1]
  File "/usr/lib/python2.7/posixpath.py", line 83, in split
    i = p.rfind('/') + 1
AttributeError: 'NoneType' object has no attribute 'rfind'

Révisions associées

Révision 7cdf8c9d (diff)
Ajouté par Joël Cuissinat il y a plus de 11 ans

  • bin/CreoleCat : erreur si template inexistant

Fixes #4538 @15m

Révision d621b029 (diff)
Ajouté par Daniel Dehennin il y a plus de 11 ans

Amélioration des tests pre-templatisation et des logs

  • creole/error.py (TemplateDisabled): Exception pour les templates
    désactivés.
  • creole/template.py (CreoleTemplateEngine._check_filevar): Vérifie que
    le dictionnaire « filevar » contient les cléfs nécessaires à la
    templatisation.
    (CreoleTemplateEngine._copy_to_template_dir): Vérification
    de « filevar ».
    (CreoleTemplateEngine.process): Idem.
    (CreoleTemplateEngine.remove_destfile): Ajout du log d’inexistance du
    fichier en mode debug.
    (CreoleTemplateEngine._instance_file): Ajout du log de l’instanciation.
    Lève l’exception TemplateDisabled si le template n’est pas actif.
    (CreoleTemplateEngine.instance_file): Suprression du log
    d’instanciation géré par « CreoleTemplateEngine._instance_file() ».
    Réécriture du log de suppression.
    (CreoleTemplateEngine.instance_files): Suppression du log
    d’instanciation géré par « CreoleTemplateEngine._instance_file() ».
    Log des templates désactivé en mode debug.

Fixes: #4538 @1h

Révision 4f83f8ea (diff)
Ajouté par Daniel Dehennin il y a environ 11 ans

Les options « -t » et « -o » de CreoleCat ne sont pas compatibles

  • bin/CreoleCat (main): Si les options « -t » et « -o » sont spécifier il
    faut renseigner la source.
    Amélioration des imports et harmonisation des messages d’exception.
  • creole/template.py (CreoleTemplateEngine._check_filevar): Amélioration
    des messages d’erreurs des raises.
    Ajout de la vérification d’existance du template.

Fixes: #4538 @5m

Révision af71625b (diff)
Ajouté par Daniel Dehennin il y a environ 11 ans

Le test d’existance des templates n’utilise pas le bon répertoire

Les templates ne sont dans « creole.config.templatedir »
qu’après « CreoleTemplateEngine._copy_file() ».

  • creole/template.py (CreoleTemplateEngine._check_filevar): Utilisation
    de « creole.config.distrib_dir »

Ref: #4538 @2m

Révision 0cef7e0f (diff)
Ajouté par moyooo il y a plus de 10 ans

All search on problems see #4538

Historique

#1 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#2 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • Assigné à changé de Daniel Dehennin à Joël Cuissinat

#3 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • Statut changé de Résolu à À valider
  • Assigné à changé de Joël Cuissinat à Daniel Dehennin
  • Version cible changé de Eole 2.4-dev-1 à Eole 2.4-dev-2
  • % réalisé changé de 100 à 50

En fait, j'ai corrigé uniquement pour CreoleCat mais il faudrait effectivement modifier quelque chose au niveau de creoled également :)

#4 Mis à jour par Daniel Dehennin il y a plus de 11 ans

creoled se moque bien de savoir si la conf est bonne, ce n’est pas son problème.

Par contre au niveau de source:creole/template.py?rev=c8fd5af5#L350 je ne dis pas… ;-)

#5 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • Statut changé de À valider à Résolu
  • % réalisé changé de 50 à 100

Appliqué par commit 7cdf8c9df4b40e0f025f256bed5d09d0953883be.

Les commit ont été rejoués suite à un nouveau plantage de la communication git/RedMine !!!

#6 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • Statut changé de Résolu à À valider

#7 Mis à jour par Joël Cuissinat il y a plus de 11 ans

  • % réalisé changé de 100 à 0

#8 Mis à jour par Daniel Dehennin il y a plus de 11 ans

  • Statut changé de À valider à Résolu
  • % réalisé changé de 0 à 100

#9 Mis à jour par Joël Cuissinat il y a environ 11 ans

  • Statut changé de Résolu à À valider
  • % réalisé changé de 100 à 80

C'est mieux mais le message d'erreur mentionne le fichier de destination au lieu de la source :

root@scribe:~# CreoleCat -t pouet.tmpl -o toto
root - Fichier template inexistant: toto

#10 Mis à jour par Daniel Dehennin il y a environ 11 ans

  • Statut changé de À valider à Résolu
  • % réalisé changé de 80 à 100

#11 Mis à jour par Fabrice Barconnière il y a environ 11 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF