general: add option to have submission channel on global listing (#14098) #813

Merged
fpeters merged 1 commits from wip/14098-submission-channel-no-welco into main 2023-11-10 08:59:13 +01:00
Owner

(instead of relying on the presence of a welco_url variable)

(instead of relying on the presence of a welco_url variable)
fpeters force-pushed wip/14098-submission-channel-no-welco from 3385cd5b57 to 9b88c43e25 2023-11-04 17:58:15 +01:00 Compare
fpeters force-pushed wip/14098-submission-channel-no-welco from 9b88c43e25 to c6101d639a 2023-11-04 19:57:55 +01:00 Compare
fpeters reviewed 2023-11-05 08:59:37 +01:00
@ -630,0 +631,4 @@
if enabled('submission-channels'):
r += htmltext('<dt><a href="submission-channels">%s</a></dt> <dd>%s</dd>') % (
_('Submission channels'),
_('Configure submission channels related options'),
Author
Owner

J'ai hésité entre ça et un écran de paramétrage "Vue globale" (j'étais d'ailleurs parti là-dessus), mais il y a demande pour d'autres paramétrages sur les canaux de saisie (voir https://dev.entrouvert.org/issues/75961) et donc ce choix.

J'ai hésité entre ça et un écran de paramétrage "Vue globale" (j'étais d'ailleurs parti là-dessus), mais il y a demande pour d'autres paramétrages sur les canaux de saisie (voir https://dev.entrouvert.org/issues/75961) et donc ce choix.
Owner

Est-ce qu'on étendrait pas à "submission" ("Saisie backoffice" d'une façon générale) au lieu de juste "submission-channels" ? Dans l'idée qu'on y piloterait un jour d'autres choses en relation avec la saisie backoffice -- ceci étant je suis bien incapable de dire quoi.

Est-ce qu'on étendrait pas à "submission" ("Saisie backoffice" d'une façon générale) au lieu de juste "submission-channels" ? Dans l'idée qu'on y piloterait un jour d'autres choses en relation avec la saisie backoffice -- ceci étant je suis bien incapable de dire quoi.
@ -1318,0 +1329,4 @@
'include-in-global-listing',
title=_('Include submission channel column in global listing'),
value=submission_channels_cfg.get('include-in-global-listing'),
)
Author
Owner

Ça démarrer avec une seule case à cocher mais l'écran servira pour d'autres options via d'autres tickets.

Ça démarrer avec une seule case à cocher mais l'écran servira pour d'autres options via d'autres tickets.
@ -617,3 +617,3 @@
return redirect('listing?' + urllib.parse.urlencode(get_request().form))
formdatas = sql.AnyFormData.select(criterias, order_by=order_by, limit=limit, offset=offset)
include_submission_channel = bool(get_publisher().get_site_option('welco_url', 'variables'))
include_submission_channel = get_cfg('submission-channels', {}).get('include-in-global-listing')
Author
Owner

Plutôt que regarder si welco_url existe, on regarde la nouvelle case à cocher.

À noter que ça fait un certain temps que welco_url n'est plus posée, même avec welco déployé (#31683, il y a 4 ans); il y aura peut-être quelques endroits qui avaient welco_url, et perdront donc la colonne avec ce patch, si ça arrive, et que quelqu'un s'en rend compte, il y aura juste à cocher la case.

Plutôt que regarder si welco_url existe, on regarde la nouvelle case à cocher. À noter que ça fait un certain temps que welco_url n'est plus posée, même avec welco déployé (#31683, il y a 4 ans); il y aura peut-être quelques endroits qui avaient welco_url, et perdront donc la colonne avec ce patch, si ça arrive, et que quelqu'un s'en rend compte, il y aura juste à cocher la case.
@ -1811,3 +1811,2 @@
yield FakeField('digest', 'digest', _('Digest'))
if get_publisher().get_site_option('welco_url', 'variables'):
yield FakeField('submission_channel', 'submission_channel', _('Channel'))
yield FakeField('submission_channel', 'submission_channel', _('Channel'))
Author
Owner

Pour les tableaux de traitement par formulaire, comme il s'agit juste pour le canal de saisie d'être proposé en possiblité de colonne (vs la vue globale où ça ajoute une colonne), ici je décide de tout le temps proposer le choix.

Pour les tableaux de traitement par formulaire, comme il s'agit juste pour le canal de saisie d'être proposé en possiblité de colonne (vs la vue globale où ça ajoute une colonne), ici je décide de tout le temps proposer le choix.
fpeters changed title from WIP: general: add option to have submission channel on global listing (#14098) to general: add option to have submission channel on global listing (#14098) 2023-11-05 08:59:44 +01:00
lguerin approved these changes 2023-11-06 10:19:26 +01:00
fpeters merged commit 4d79167e6a into main 2023-11-10 08:59:13 +01:00
fpeters deleted branch wip/14098-submission-channel-no-welco 2023-11-10 08:59:13 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
3 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#813
No description provided.