Project

General

Profile

Anomalie #4538

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

Added by Emmanuel GARETTE about 7 years ago. Updated almost 7 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Category:
-
Start date:
12/03/2012
Due date:
% Done:

100%

Spent time:
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'

Associated revisions

Revision 7cdf8c9d (diff)
Added by Joël Cuissinat about 7 years ago

  • bin/CreoleCat : erreur si template inexistant

Fixes #4538 @15m

Revision d621b029 (diff)
Added by Daniel Dehennin about 7 years ago

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

Revision 4f83f8ea (diff)
Added by Daniel Dehennin almost 7 years ago

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

Revision af71625b (diff)
Added by Daniel Dehennin almost 7 years ago

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

Revision 0cef7e0f (diff)
Added by moyooo over 6 years ago

All search on problems see #4538

History

#1 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#2 Updated by Joël Cuissinat about 7 years ago

  • Assigned To changed from Daniel Dehennin to Joël Cuissinat

#3 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Résolu to Réouvert
  • Assigned To changed from Joël Cuissinat to Daniel Dehennin
  • Target version changed from Eole 2.4-dev-1 to Eole 2.4-dev-2
  • % Done changed from 100 to 50

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

#4 Updated by Daniel Dehennin about 7 years ago

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 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Réouvert to Résolu
  • % Done changed from 50 to 100

Appliqué par commit 7cdf8c9df4b40e0f025f256bed5d09d0953883be.

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

#6 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Résolu to Réouvert

#7 Updated by Joël Cuissinat about 7 years ago

  • % Done changed from 100 to 0

#8 Updated by Daniel Dehennin about 7 years ago

  • Status changed from Réouvert to Résolu
  • % Done changed from 0 to 100

#9 Updated by Joël Cuissinat almost 7 years ago

  • Status changed from Résolu to Réouvert
  • % Done changed from 100 to 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 Updated by Daniel Dehennin almost 7 years ago

  • Status changed from Réouvert to Résolu
  • % Done changed from 80 to 100

#11 Updated by Fabrice Barconnière almost 7 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF