entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

workflows: add option to get document model file using a template (#69689) #1177

Merged
fpeters merged 1 commits from wip/69689-doc-template-from-template into main 2024-02-29 10:44:23 +01:00
Conversation 0 Commits 1 Files Changed 6 +202 -66
fpeters commented 2024-02-21 16:58:28 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/69689-doc-template-from-template from 8e88832d2b to f22f9a3be2 2024-02-21 17:19:05 +01:00 Compare
fpeters force-pushed wip/69689-doc-template-from-template from f22f9a3be2 to 42c6492959 2024-02-21 17:40:00 +01:00 Compare
fpeters force-pushed wip/69689-doc-template-from-template from 42c6492959 to bca153db9b 2024-02-21 17:48:04 +01:00 Compare
fpeters force-pushed wip/69689-doc-template-from-template from bca153db9b to 384f0a6408 2024-02-21 17:56:02 +01:00 Compare
fpeters force-pushed wip/69689-doc-template-from-template from 384f0a6408 to 60d8cd41d0 2024-02-22 09:24:34 +01:00 Compare
fpeters reviewed 2024-02-22 09:44:54 +01:00
wcs/wf/export_to_model.py Outdated
@ -415,4 +428,0 @@
if formdef:
hint = htmltext('%s: <ul class="varnames">') % _('Available variables')
varnames = get_varnames(formdef.fields)
for pair in varnames:
fpeters commented 2024-02-22 09:43:07 +01:00
Author
Owner
Copy Link

Il y avait cette partie sur le cas très particulier de l'édition de ce paramètre via l'ancienné mécanique d'option de workflow (qui s'édite depuis la page d'un formulaire, où on a donc un formdef), où on affichait les variables exposées (mais avec les anciens noms qui marchaient uniquement avec python). Bref je retire ça.

Il y avait cette partie sur le cas très particulier de l'édition de ce paramètre via l'ancienné mécanique d'option de workflow (qui s'édite depuis la page d'un formulaire, où on a donc un formdef), où on affichait les variables exposées (mais avec les anciens noms qui marchaient uniquement avec python). Bref je retire ça.
wcs/wf/export_to_model.py Outdated
@ -281,2 +283,4 @@
label = None
model_file_mode = 'file' # or 'template'
model_file = None
model_file_template = None
fpeters commented 2024-02-22 09:39:56 +01:00
Author
Owner
Copy Link

Un attribut model_file_mode qui est 'file' ou 'template', et dessous model_file qui est le fichier directement uploadé comme aujourd'hui, et model_file_template qui est la nouveauté.

Un attribut model_file_mode qui est 'file' ou 'template', et dessous model_file qui est le fichier directement uploadé comme aujourd'hui, et model_file_template qui est la nouveauté.
wcs/workflows.py Outdated
@ -3175,2 +3175,3 @@
r += htmltext('<li class="parameter-%s">' % parameter)
r += htmltext('<span class="parameter">%s</span> ') % _('%s:') % widget.get_title()
if widget.get_title():
r += htmltext('<span class="parameter">%s</span> ') % _('%s:') % widget.get_title()
fpeters commented 2024-02-22 09:44:49 +01:00
Author
Owner
Copy Link

Dans l'inspect du workflow, pour l'affichage des paramètres, avec model_file/model_file_template on n'a pas de titre, ça faisait un ":" vide.

Dans l'inspect du workflow, pour l'affichage des paramètres, avec model_file/model_file_template on n'a pas de titre, ça faisait un ":" vide.
fpeters force-pushed wip/69689-doc-template-from-template from 60d8cd41d0 to 33e79bd320 2024-02-22 09:45:01 +01:00 Compare
fpeters force-pushed wip/69689-doc-template-from-template from 33e79bd320 to 715de81161 2024-02-22 09:53:26 +01:00 Compare
fpeters changed title from WIP: workflows: add option to get document model file using a template (#69689) to workflows: add option to get document model file using a template (#69689) 2024-02-22 10:22:50 +01:00
lguerin reviewed 2024-02-22 12:26:46 +01:00
wcs/wf/export_to_model.py Outdated
@ -292,3 +278,3 @@
def get_line_details(self):
if self.model_file:
if self.model_file and self.model_file_mode == 'file':
lguerin commented 2024-02-22 12:23:10 +01:00
Owner
Copy Link

pour l'existant, model_file_mode vaut None, non ?

pour l'existant, model_file_mode vaut None, non ?
fpeters commented 2024-02-26 12:58:41 +01:00
Author
Owner
Copy Link

Il ne sera pas défini sur l'objet sérialisé donc ça tombera sur la valeur posée au niveau de la classe.

Il ne sera pas défini sur l'objet sérialisé donc ça tombera sur la valeur posée au niveau de la classe.
fpeters force-pushed wip/69689-doc-template-from-template from 715de81161 to 03226fdd38 2024-02-26 12:56:56 +01:00 Compare
lguerin approved these changes 2024-02-27 15:15:24 +01:00
fpeters merged commit f355b9ca02 into main 2024-02-29 10:44:23 +01:00
fpeters deleted branch wip/69689-doc-template-from-template 2024-02-29 10:44:23 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#1177
No description provided.
Delete Branch "wip/69689-doc-template-from-template"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?