Bug #3928
Les variables de site_options reçues ne sont pas exploitables par string.Template
0%
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.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 10 ans
- Fichier 0001-flatten-site-options-so-they-can-be-used-in-string-t.patch 0001-flatten-site-options-so-they-can-be-used-in-string-t.patch ajouté
- Statut changé de Nouveau à En cours
L'idée est du coup de bêtement aplatir le dictionnaire.
Quelqu'un pour relire ?
Mis à jour par Jérôme Schneider il y a plus de 10 ans
Pour moi c'est bon. Le flatten_dict fonctionne correctement. Benj j'attends ton go avant de le pousser dans le dépôt.
Mis à jour par Benjamin Dauvergne il y a plus de 10 ans
- Fichier 0001-deploy-use-str.format-instead-of-string.Template.patch 0001-deploy-use-str.format-instead-of-string.Template.patch ajouté
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.
Mis à jour par Benjamin Dauvergne il y a plus de 10 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit 4b21e65ea6f568d444d8c8e0b53b3f0858cb90a0.
Mis à jour par Benjamin Dauvergne il y a plus de 10 ans
- Statut changé de Résolu (à déployer) à En cours
- % réalisé changé de 100 à 0
Je vais pousser mon patch puis ouvrir un ticket pour Jérôme de mise à jour des squelettes.
Mis à jour par Benjamin Dauvergne il y a plus de 10 ans
- Statut changé de En cours à Solution déployée
Déployé en dév, j'ai fait la modif aux templates manuellement.
deploy: use str.format instead of string.Template
fixes #3928