Development #32495
import_template spécifique pour combo
0%
Description
On a donné une signification particulière à "portal-user" et "portal-agent" dans template_name; un appel demandant ces noms devrait juste être zappé. (parce que sinon on voit défiler des messages "unknown template" sur stderr et ça nous fera rater de vraies erreurs.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-agent-ship-a-custom-import_template-for-combo-to-exc.patch 0001-agent-ship-a-custom-import_template-for-combo-to-exc.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Christophe Siraut il y a presque 5 ans
Pourquoi utiliser template_name pour deux usages différents? On pourrait vouloir créer un portal-user + importer un gabarit, et donc avoir besoin de 2 variables distinctes, non?
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-combo-add-custom-import_template-to-ignore-errors-wi.patch 0001-combo-add-custom-import_template-to-ignore-errors-wi.patch ajouté
Il y a un usage déjà effectif aujourd'hui, avec template_name qui contient portal-user ou portal-agent, l'autre usage existant aujourd'hui c'est côté wcs, où c'est un fichier zip qui sert de modèle.
Dans #14630 il a été décidé d'utiliser le template_name existant pour également aller chercher des fichiers, et l'implémentation par défaut va chercher des fichiers, json.
Côté w.c.s. ça ne pose pas de problème parce que le hobo_deploy est géré de manière totalement indépendante, côté combo, c'est le code générique qui est utilisé, code générique qui ne peut pas avoir connaissance de cette spécificité.
Mais autre approche, pour se permettre d'avancer s'il existait réellement un portal-agent.json.
Mis à jour par Christophe Siraut il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 362af52cdda0efe33874b9c007831eab396be7c4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Apr 19 14:52:12 2019 +0200 combo: add custom import_template to ignore errors with some names (#32495)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
combo: add custom import_template to ignore errors with some names (#32495)