misc: make timetable trigger change event (#82042) #769

fpeters · 2023-10-07T17:41:12+02:00

fpeters commented

2023-10-07 17:41:12 +02:00

Cette partie sur les évènements js déclenchés sur des changements de données reste confuse, plus aucun souvenir de l'objectif de distinction entre "wcs:change" et "change", à un moment peut-être le "wcs:change" concerné par le formulaire vs les "change" sur les champs individuels, mais ça n'est plus ça, il y a du wcs:change sur des champs individuels, et tout ça doit rebondir d'objets en objets c'est pas clair.

Ici je modifie l'événement envoyé après qu'on ait modifié la valeur contenue dans le select caché d'un champ de sélection horaire; c'est ainsi attrapé par :

        form.querySelectorAll(inputsSelector).forEach((input) => {
          ['change', 'input', 'paste', 'wcs:change'].forEach((eventName) => {
            $(input).on(eventName, () => {
              const params = {};
              params['modified_field'] = input.closest('[data-field-id]').dataset.fieldId
...
              $(form).trigger('wcs:change', params)

et le modified_field_id arrive in fine en paramètre de l'appel à l'adresse à /live.

Le code comme écrit là on dirait que l'événement "wcs:change" devrait fonctionner pareil et peut-être que oui mais ça finaissait quand même avec un événement en plus (?) sur le formulaire (le "wcs:change" envoyé sur le select qui y descendrait, en plus du wcs:change envoyé ici ?).

Seule certitute c'est que sur le petit cas particulier ici, le paramètre qui n'était pas envoyé l'est désormais.

(mais peut-être que ça fait s'écrouler d'autres trucs)

Cette partie sur les évènements js déclenchés sur des changements de données reste confuse, plus aucun souvenir de l'objectif de distinction entre "wcs:change" et "change", à un moment peut-être le "wcs:change" concerné par le formulaire vs les "change" sur les champs individuels, mais ça n'est plus ça, il y a du wcs:change sur des champs individuels, et tout ça doit rebondir d'objets en objets c'est pas clair. Ici je modifie l'événement envoyé après qu'on ait modifié la valeur contenue dans le select caché d'un champ de sélection horaire; c'est ainsi attrapé par : ``` form.querySelectorAll(inputsSelector).forEach((input) => { ['change', 'input', 'paste', 'wcs:change'].forEach((eventName) => { $(input).on(eventName, () => { const params = {}; params['modified_field'] = input.closest('[data-field-id]').dataset.fieldId ... $(form).trigger('wcs:change', params) ``` et le modified_field_id arrive in fine en paramètre de l'appel à l'adresse à /live. Le code comme écrit là on dirait que l'événement "wcs:change" devrait fonctionner pareil et peut-être que oui mais ça finaissait quand même avec un événement en plus (?) sur le formulaire (le "wcs:change" envoyé sur le select qui y descendrait, en plus du wcs:change envoyé ici ?). Seule certitute c'est que sur le petit cas particulier ici, le paramètre qui n'était pas envoyé l'est désormais. (mais peut-être que ça fait s'écrouler d'autres trucs)

fpeters added 1 commit 2023-10-07 17:41:13 +02:00

gitea/wcs/pipeline/head This commit looks good Details

b8e3393518 misc: make timetable trigger change event (#82042 )

ecazenave commented

2023-10-09 10:21:56 +02:00

Je valide vite dans l'idée de bénéficier d'une longue période de recette.

ecazenave approved these changes 2023-10-09 10:22:03 +02:00

fpeters commented

2023-10-09 11:27:24 +02:00

Je valide vite dans l'idée de bénéficier d'une longue période de recette.

Perso je comptais ne rien pousser/tagguer/déployer dans w.c.s. cette semaine.

> Je valide vite dans l'idée de bénéficier d'une longue période de recette. Perso je comptais ne rien pousser/tagguer/déployer dans w.c.s. cette semaine.

fpeters merged commit 8da1f605b2 into main

2023-10-27 08:35:28 +02:00

fpeters deleted branch wip/82042-timetable-trigger-change

2023-10-27 08:35:28 +02:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

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#769