Project

General

Profile

Bug #3928

Les variables de site_options reçues ne sont pas exploitables par string.Template

Added by Frédéric Péters over 10 years ago. Updated over 10 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
31 October 2013
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Planning:

Description

le site_options reçu ressemblerait à {'drupal': False, 'variables': {'key', 'value'}} mais il me semble que ça n'est pas compatible avec le string.Template utilisé pour générer le fichier site_options.cfg.


Files

Associated revisions

Revision 4b21e65e (diff)
Added by Benjamin Dauvergne over 10 years ago

deploy: use str.format instead of string.Template

fixes #3928

History

#1

Updated by Frédéric Péters over 10 years ago

L'idée est du coup de bêtement aplatir le dictionnaire.

Quelqu'un pour relire ?

#2

Updated by Jérôme Schneider over 10 years ago

Pour moi c'est bon. Le flatten_dict fonctionne correctement. Benj j'attends ton go avant de le pousser dans le dépôt.

#3

Updated by Benjamin Dauvergne over 10 years ago

On peut aussi utiliser str.format au lieu de string.Template. La prolifération des syntaxes de template devient inquiétante :)
Mais dans ce cas il faut aussi corriger les squelettes de site_options.cfg, des fichiers de métadonnées SAML 2.0 et le virtual host apache 2.0. Donc dans un premier temps ok pour le patch de Fred, mais ce serait bien de passer à une autre syntaxe après et aussi de packager/versionner le squelette de site qu'on puisse modifier ces fichiers squelettes facilement.

#4

Updated by Benjamin Dauvergne over 10 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 0 to 100
#5

Updated by Benjamin Dauvergne over 10 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 0

Je vais pousser mon patch puis ouvrir un ticket pour Jérôme de mise à jour des squelettes.

#6

Updated by Benjamin Dauvergne over 10 years ago

  • Status changed from En cours to Solution déployée

Déployé en dév, j'ai fait la modif aux templates manuellement.

Also available in: Atom PDF