Projet

Général

Profil

Development #19188

ajouter une option "envoyer dans un champ backoffice" sur l'action de génération d'un document

Ajouté par Thomas Noël il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Présenter une liste des champs de traitement de type fichier. L'option ne sera pas présentée si aucun champ de ce type n'existe.

C'est pour profiter plus facilement le #8274, ie ça serait bien que ça vienne dans la même release.


Fichiers


Demandes liées

Lié à w.c.s. - Development #13806: Action génération de document : stocker dans un champs de traitementFermé31 octobre 2016

Actions

Révisions associées

Révision eb586a7c (diff)
Ajouté par Thomas Noël il y a plus de 6 ans

workflow: re-order export to model UI and code (#19188)

Révision 2b193960 (diff)
Ajouté par Thomas Noël il y a plus de 6 ans

workflow: allow to send generated docs in backoffice field (#19188)

Historique

#1

Mis à jour par Thomas Noël il y a plus de 6 ans

0001 pour remettre les options de l'action un peu dans un ordre logique (par exemple parce que varname n'est utile que dans le cas attach_to_history.
0002 l'ajout de l'enregistrement dans une variable backoffice (s'il y en a au moins une de type 'file')

#2

Mis à jour par Frédéric Péters il y a plus de 6 ans

0001

Mais il n'était pas nécessaire de déplacer get_parameters, il me semble.

title=_('Store in a Backoffice File Field'),

On ne capitalise pas ainsi les autres libellés.

outstream.read()

Je verrais bien un .seek(0) avant de consommer le contenu (ce n'est pas facile à suivre (raison principale) mais AttachmentEvolutionPart me semble pouvoir lire le fichier jusqu'au bout avant que ce code ne soit appelé).

#3

Mis à jour par Thomas Noël il y a plus de 6 ans

Frédéric Péters a écrit :

0001

Mais il n'était pas nécessaire de déplacer get_parameters, il me semble.

En fait j'avais dit dans le commit "workflow: re-order export to model UI and code", parce qu'un peu partout ailleurs get_parameters est au dessus. Et c'était pour rendre le patch suivant plus lisible. Donc je maintiens le 0001 tel quel, parce que j'ai la flemme et cet argument.

title=_('Store in a Backoffice File Field'),

On ne capitalise pas ainsi les autres libellés.

Arf, oui.

outstream.read()

Je verrais bien un .seek(0) avant de consommer le contenu (ce n'est pas facile à suivre (raison principale) mais AttachmentEvolutionPart me semble pouvoir lire le fichier jusqu'au bout avant que ce code ne soit appelé).

Bien vu, j'ai ajouté et modifié le test pour vérifier que quand on cumule les deux actions (enregistrer dans l'historique et dans le champ de traitement) ça se passe bien ; et ça ne passait pas sans le seek(0).

#4

Mis à jour par Frédéric Péters il y a plus de 6 ans

Ack.

#5

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 2b193960e93ebbb69e6318f0073c2540e6626b64
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Oct 4 11:40:47 2017 +0200

    workflow: allow to send generated docs in backoffice field (#19188)

commit eb586a7cfafc36d1d1c8bc890d422be6456a12fe
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed Oct 4 09:09:29 2017 +0200

    workflow: re-order export to model UI and code (#19188)

#6

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

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Frédéric Péters il y a 7 mois

  • Lié à Development #13806: Action génération de document : stocker dans un champs de traitement ajouté

Formats disponibles : Atom PDF