Skip to content

Conversation

@hellochloe1989
Copy link
Contributor

@hellochloe1989 hellochloe1989 commented Sep 30, 2025

Ce commit modifie le parsing des query string des url des documents générés pour les champs de type checkbox (actuellement la valeur des checkbox est parsée en tant que string 'false' ou 'true', ce qui est toujours coercé en booléen true au rendu).

Si la valeur dans la query string est 'false', '0' ou '' (chaine vide) la valeur parsée sera le booléen false.
Si la valeur dans la query string est autre chose la valeur parsée sera le booléen true.
Cela imite le constructeur de Boolean.

Le parsing des autres types de champs n'est pas impacté.

Procédure de test :

Fix #61

@hellochloe1989 hellochloe1989 marked this pull request as ready for review September 30, 2025 18:28
@hellochloe1989
Copy link
Contributor Author

Testé en dev. On peut revoir la logique de parsing si besoin !

@hellochloe1989 hellochloe1989 force-pushed the fix-parse-boolean-in-query branch from 37ef210 to c696918 Compare September 30, 2025 18:36
Copy link
Contributor

@agateblue agateblue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le code fait sens, j'ai testé localement et ça fonctionne comme prévu, merci !

@agateblue agateblue merged commit 79b78db into Fransgenre:main Oct 1, 2025
1 check passed
@hellochloe1989 hellochloe1989 deleted the fix-parse-boolean-in-query branch October 1, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Parsing incorrect des checkbox dans les liens de partage des documents générés

2 participants