Projet

Général

Profil

Development #32495

import_template spécifique pour combo

Ajouté par Frédéric Péters il y a presque 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
19 avril 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 362af52c (diff)
Ajouté par Frédéric Péters il y a presque 5 ans

combo: add custom import_template to ignore errors with some names (#32495)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 5 ans

#2

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?

#3

Mis à jour par Frédéric Péters il y a presque 5 ans

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.

#4

Mis à jour par Christophe Siraut il y a presque 5 ans

  • Statut changé de Solution proposée à Solution validée
#5

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)
#6

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

Formats disponibles : Atom PDF