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

workflows: include varname for field options with duplicated labels (#60315) #1050

Merged
fpeters merged 1 commits from wip/60315-field-select-varname into main 2024-01-26 08:46:24 +01:00
Conversation 0 Commits 1 Files Changed 4 +107 -8
fpeters commented 2024-01-20 10:24:08 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/60315-field-select-varname from ae555f5dbf to 4913a72cd3 2024-01-20 10:55:23 +01:00 Compare
fpeters reviewed 2024-01-21 09:51:37 +01:00
wcs/wf/backoffice_fields.py Outdated
@ -48,0 +55,4 @@
continue
label = f'{field.label} - {field.get_type_label()}'
if label in repeated_labels and field.varname:
label = f'{label} ({field.varname})'
fpeters commented 2024-01-21 09:49:39 +01:00
Author
Owner
Copy Link

Ajout de l’identifiant uniquement s’il a des doublons sur les libellés, ça devrait laisser l’interface identique à l’actuelle la plupart du temps.

Ajout de l’identifiant uniquement s’il a des doublons sur les libellés, ça devrait laisser l’interface identique à l’actuelle la plupart du temps.
wcs/wf/create_formdata.py Outdated
@ -77,3 +84,3 @@
options = [(None, '---', '')]
for field in formdef.get_widget_fields():
options.append((field.id, field.label, str(field.id)))
label = f'{field.label} - {field.get_type_label()}'
fpeters commented 2024-01-21 09:50:24 +01:00
Author
Owner
Copy Link

Dans les actions de création de demande/fiche, on n’avait pas encore fait la modification pour afficher le type de champ, c’est intégré ici.

Dans les actions de création de demande/fiche, on n’avait pas encore fait la modification pour afficher le type de champ, c’est intégré ici.
wcs/wf/create_formdata.py Outdated
@ -80,0 +87,4 @@
block_label = field.label
if label in repeated_labels and field.varname:
label = f'{label} ({field.varname})'
block_label = f'{field.label} ({field.varname})'
fpeters commented 2024-01-21 09:50:49 +01:00
Author
Owner
Copy Link

Puis on fait la même affaire, s’il y a des doublons, ajouter l’identifiant.

Puis on fait la même affaire, s’il y a des doublons, ajouter l’identifiant.
wcs/wf/create_formdata.py Outdated
@ -83,3 +95,3 @@
(
f'{field.id}${subfield.id}',
f'{field.label} - {subfield.label}',
f'{block_label} - {subfield.label} - {subfield.get_type_label()}',
fpeters commented 2024-01-21 09:51:33 +01:00
Author
Owner
Copy Link

Pour les blocs j’ai aussi expérimenté avec un <optgroup> mais le résultat n’était pas terrible, donc je laisse juste ainsi.

Pour les blocs j’ai aussi expérimenté avec un `<optgroup>` mais le résultat n’était pas terrible, donc je laisse juste ainsi.
fpeters changed title from WIP: workflows: include varname for field options with duplicated labels (#60315) to workflows: include varname for field options with duplicated labels (#60315) 2024-01-21 09:51:44 +01:00
fpeters force-pushed wip/60315-field-select-varname from 4913a72cd3 to aee429345a 2024-01-22 14:22:17 +01:00 Compare
lguerin approved these changes 2024-01-22 14:29:30 +01:00
fpeters force-pushed wip/60315-field-select-varname from aee429345a to 233adfd44b 2024-01-22 14:35:53 +01:00 Compare
fpeters merged commit 6a175aa5de into main 2024-01-26 08:46:24 +01:00
fpeters deleted branch wip/60315-field-select-varname 2024-01-26 08:46:24 +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#1050
No description provided.
Delete Branch "wip/60315-field-select-varname"

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?