Valeurs '0' transformées en chaines vides dans les inputs #225

Open
opened 1 week ago by nicod_ · 6 comments
nicod_ commented 1 week ago
Collaborator

Depuis l'introduction de saisies_utf8_restaurer_planes(), une valeur 0 passée à un input est transformé en chaine vide :
777a7b9539/saisies_fonctions.php (L309)

En retournant simplement $x au lieu de '' ça retombe en marche.

Depuis l'introduction de saisies_utf8_restaurer_planes(), une valeur 0 passée à un input est transformé en chaine vide : https://git.spip.net/spip-contrib-extensions/saisies/src/commit/777a7b95397627f28e9f2e32fba0d673ab50d53d/saisies_fonctions.php#L309 En retournant simplement $x au lieu de '' ça retombe en marche.
Owner

Corrigé en 07d4a45 donc.

Corrigé en 07d4a45 donc.
maieul closed this issue 3 days ago
Poster
Collaborator

Merde, ça génère une erreur à cause du typage :

Erreur d’exécution plugins/spip-contrib-extensions/saisies/saisies/input.html | File […]/plugins/spip-contrib-extensions/saisies/saisies_fonctions.php Line 310 : Return value of saisies_utf8_restaurer_planes() must be of the type string, null returned

Merde, ça génère une erreur à cause du typage : > Erreur d’exécution plugins/spip-contrib-extensions/saisies/saisies/input.html | File […]/plugins/spip-contrib-extensions/saisies/saisies_fonctions.php Line 310 : Return value of saisies_utf8_restaurer_planes() must be of the type string, null returned
nicod_ reopened this issue 3 days ago
Poster
Collaborator

Je propose d'ajouter un cast spécifique : return (string)$x;

Je propose d'ajouter un cast spécifique : `return (string)$x;`
Poster
Collaborator

Je vois que tu n'as pas encore fait (ou poussé) le tag.
J'ai corrigé directement : 917d745551

Je vois que tu n'as pas encore fait (ou poussé) le tag. J'ai corrigé directement : https://git.spip.net/spip-contrib-extensions/saisies/commit/917d745551ee54f5bf627cb5250ddc7eaca972a4
Owner

mince oui. merci pour le commit. Du coup j'hésite à taguer pour l'instant. Je te laisse faire dans quelque jour quand on sera sur qu'on a rien loupé ?

mince oui. merci pour le commit. Du coup j'hésite à taguer pour l'instant. Je te laisse faire dans quelque jour quand on sera sur qu'on a rien loupé ?
Poster
Collaborator

Je n'avais que cette erreur là, pas d'autre, ce qui est logique.
Et les valeurs '0' remontent bien comme telles.
Mais ça ne presse pas effectivement.

Je n'avais que cette erreur là, pas d'autre, ce qui est logique. Et les valeurs '0' remontent bien comme telles. Mais ça ne presse pas effectivement.
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.