Project

General

Profile

Development #32495

import_template spécifique pour combo

Added by Frédéric Péters about 5 years ago. Updated about 5 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
19 April 2019
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 362af52c (diff)
Added by Frédéric Péters about 5 years ago

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

History

#1

Updated by Frédéric Péters about 5 years ago

#2

Updated by Christophe Siraut about 5 years ago

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

Updated by Frédéric Péters about 5 years ago

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

Updated by Christophe Siraut about 5 years ago

  • Status changed from Solution proposée to Solution validée
#5

Updated by Frédéric Péters about 5 years ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters about 5 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF