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

misc: add special support for tuple/list/set data in query strings (#85776) #1041

Merged
fpeters merged 1 commits from wip/85776-complex-qsdata into main 2024-02-15 10:11:28 +01:00
Conversation 2 Commits 1 Files Changed 2 +103 -6
fpeters commented 2024-01-17 16:00:58 +01:00
Owner
Copy Link

Avec ma préférence pour que ça produise key=v1,v2, plutôt que key=v1&key=v2, mais ça peut se discuter un peu.

Avec ma préférence pour que ça produise key=v1,v2, plutôt que key=v1&key=v2, mais ça peut se discuter un peu.
fpeters added 1 commit 2024-01-17 16:00:58 +01:00
tnoel reviewed 2024-01-18 12:00:42 +01:00
wcs/wscalls.py Outdated
@ -126,0 +122,4 @@
if value:
value = get_publisher().get_cached_complex_data(value)
if isinstance(value, (tuple, list, set)):
qs.append((key, ','.join(value)))
tnoel commented 2024-01-18 12:00:34 +01:00
Owner
Copy Link

Je suis vraiment pas fan de cette nouvelle magie cachée, qui va en plus planter si un des éléments contient une virgule. L'usage de join lors de l'appel me parait plus simple, sans magie.

Je suis vraiment pas fan de cette nouvelle magie cachée, qui va en plus planter si un des éléments contient une virgule. L'usage de join lors de l'appel me parait plus simple, sans magie.
fpeters commented 2024-01-24 18:46:01 +01:00
Author
Owner
Copy Link

Mais la situation actuelle avec le str() de la liste n'est jamais utile. Avec ce ticket, que ça soit en x=a,b ou x=a&x=b, on produit quelque chose qui pourra l'être.

Et il sera toujours temps d'utiliser |join quand on veut ce comportement; ce qui est d'ailleurs un argument pour la forme x=a&x=b, qu'autrement on ne peut pas produire.

(j'ai posé un commit pour la version qui produit x=a&x=b).

Mais la situation actuelle avec le str() de la liste n'est jamais utile. Avec ce ticket, que ça soit en x=a,b ou x=a&x=b, on produit quelque chose qui pourra l'être. Et il sera toujours temps d'utiliser |join quand on veut ce comportement; ce qui est d'ailleurs un argument pour la forme x=a&x=b, qu'autrement on ne peut pas produire. (j'ai posé un commit pour la version qui produit x=a&x=b).
fpeters added 1 commit 2024-01-24 18:35:45 +01:00
gitea/wcs/pipeline/head This commit looks good Details
fdf5c5bc61 fixup, version with multiple identical keys
fpeters force-pushed wip/85776-complex-qsdata from fdf5c5bc61 to 09ca436cbe 2024-02-14 19:27:02 +01:00 Compare
fpeters commented 2024-02-14 19:31:14 +01:00
Author
Owner
Copy Link

Et il sera toujours temps d'utiliser |join quand on veut ce comportement; ce qui est d'ailleurs un argument pour la forme x=a&x=b, qu'autrement on ne peut pas produire.

Là-dessus j'ai donc laissé en unique commit la version qui créera plusieurs paramètres en cas de liste de plusieurs éléments.

Par rapport à la situation existante, qui produirait un paramètre xxx=%5B%27pomme%27%2C%20%27poire%27%5D, ce ticket est pour moi incontestablement un progrès et à valider.

> Et il sera toujours temps d'utiliser |join quand on veut ce comportement; ce qui est d'ailleurs un argument pour la forme x=a&x=b, qu'autrement on ne peut pas produire. Là-dessus j'ai donc laissé en unique commit la version qui créera plusieurs paramètres en cas de liste de plusieurs éléments. Par rapport à la situation existante, qui produirait un paramètre xxx=%5B%27pomme%27%2C%20%27poire%27%5D, ce ticket est pour moi incontestablement un progrès et à valider.
fpeters requested review from vdeniaud 2024-02-14 19:31:22 +01:00
fpeters commented 2024-02-14 19:32:04 +01:00
Author
Owner
Copy Link

@vdeniaud je te taggue comme relecteur mais il n'y a pas d'urgence je pense et on peut préférer attendre @tnoel.

@vdeniaud je te taggue comme relecteur mais il n'y a pas d'urgence je pense et on peut préférer attendre @tnoel.
fpeters changed title from WIP: misc: add special support for tuple/list/set data in query strings (#85776) to misc: add special support for tuple/list/set data in query strings (#85776) 2024-02-14 19:32:36 +01:00
vdeniaud approved these changes 2024-02-15 10:08:56 +01:00
vdeniaud left a comment
Owner
Copy Link

J'avais raté le commentaire d'il y a 3 semaines, go comme ça

J'avais raté le commentaire d'il y a 3 semaines, go comme ça
tests/workflow/test_wscall.py
@ -1047,0 +1114,4 @@
'empty_string': '{{ form_var_empty }}',
'none': '{{ form_var_none }}',
'bool': '{{ form_var_bool_raw }}',
'time': '{{ "13:12"|time }}',
vdeniaud commented 2024-02-15 10:08:43 +01:00
Owner
Copy Link

Un test de qualité !

Un test de qualité !
fpeters merged commit 8ced65d3e8 into main 2024-02-15 10:11:28 +01:00
fpeters deleted branch wip/85776-complex-qsdata 2024-02-15 10:11:28 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1041
No description provided.
Delete Branch "wip/85776-complex-qsdata"

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?