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 11 years ago. Updated over 11 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Frédéric Péters
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

0001-flatten-site-options-so-they-can-be-used-in-string-t.patch (1.7 KB) 0001-flatten-site-options-so-they-can-be-used-in-string-t.patch Frédéric Péters, 31 October 2013 02:31 PM
0001-deploy-use-str.format-instead-of-string.Template.patch (2.66 KB) 0001-deploy-use-str.format-instead-of-string.Template.patch Benjamin Dauvergne, 04 November 2013 10:37 AM

Associated revisions

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

deploy: use str.format instead of string.Template

fixes #3928

History

#1

Updated by Frédéric Péters over 11 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 11 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 11 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 11 years ago

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

Updated by Benjamin Dauvergne over 11 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 11 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