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

forms: allow dynamic prefill of autocomplete fields (#60412) #792

Open
fpeters wants to merge 1 commits from wip/60412-dynamic-prefill-select2-card into main
pull from: wip/60412-dynamic-prefill-select2-card
merge into: entrouvert:main
Conversation 2 Commits 1 Files Changed 3 +59
fpeters commented 2023-10-27 18:42:56 +02:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-10-27 18:42:56 +02:00
fpeters changed title from WIP: forms: allow dynamic prefill of autocomplete fields (#60412) to forms: allow dynamic prefill of autocomplete fields (#60412) 2023-10-27 19:54:17 +02:00
nroche commented 2023-10-30 11:18:38 +01:00
Owner
Copy Link

J'ai testé avec ce formulaire et j'ai l'impression que le patch ne corrige pas #60412.

Dans le formulaire ci-joint, j'ai pris une source de donnée Parsifal pour tester, parce qu'avec les sources de données avec valeur JSON, j'ai l'impression que le pré-remplissage ne fonctionne pas, j'ai cependant remis cette source de donnée pour pouvoir exporter le formulaire.

Dans ce formulaire, si je positionne les 2 champs sur une même source de donnée (mais pas une sources de données avec valeur JSON) et que je décoche l'autocomplétion dans la seconde liste, alors choisir un item dans la première liste, le choisi également dans la seconde.

J'ai testé avec ce formulaire et j'ai l'impression que le patch ne corrige pas #60412. Dans le formulaire ci-joint, j'ai pris une source de donnée Parsifal pour tester, parce qu'avec les sources de données avec valeur JSON, j'ai l'impression que le pré-remplissage ne fonctionne pas, j'ai cependant remis cette source de donnée pour pouvoir exporter le formulaire. Dans ce formulaire, si je positionne les 2 champs sur une même source de donnée (mais pas une sources de données avec valeur JSON) et que je décoche l'autocomplétion dans la seconde liste, alors choisir un item dans la première liste, le choisi également dans la seconde.
form-test-60412.wcs
2.7 KiB
fpeters added 1 commit 2023-10-30 12:36:25 +01:00
gitea/wcs/pipeline/head There was a failure building this commit Details
388eb9dc18 fixup
fpeters commented 2023-10-30 12:39:27 +01:00
Author
Owner
Copy Link

Je n'arrive pas à comprendre ce que tu as testé mais concernant le formulaire attaché, c'est qu'on n'est pas sur du jsonp pour la sélection/recherche de l'élément, on est sur un <select> qui contient déjà tous les éléments. Pour ce cas-là il manquait une notification à select2 comme quoi la valeur du select natif avait été modifiée, c'est fait dans le commit que je viens d'ajouter (388eb9dc18)

Je n'arrive pas à comprendre ce que tu as testé mais concernant le formulaire attaché, c'est qu'on n'est pas sur du jsonp pour la sélection/recherche de l'élément, on est sur un `<select>` qui contient déjà tous les éléments. Pour ce cas-là il manquait une notification à select2 comme quoi la valeur du select natif avait été modifiée, c'est fait dans le commit que je viens d'ajouter (https://git.entrouvert.org/entrouvert/wcs/commit/388eb9dc183e1a79a30e97cb1531f605ba7d4590)
fpeters force-pushed wip/60412-dynamic-prefill-select2-card from 388eb9dc18 to 7fbf6f6ef9 2024-04-26 08:57:29 +02:00 Compare
All checks were successful
gitea/wcs/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
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#792
No description provided.
Delete Branch "wip/60412-dynamic-prefill-select2-card"

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?