Projet

Général

Profil

Bug #47974

les sources JSON qui renvoient des id sous forme d'entiers ne fonctionnement plus en autocomplétion

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
-
Début:
22 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On a notamment un connecteur BAN "streets" qui fonctionne ainsi, et renvoie des informations comme :

id    39950
text    "11ème Rue Cité Berliet" 
type    "street" 
city    "Saint-Priest" 
citycode    "69290" 
zipcode    "69800" 

où id est donc un entier.

Quand on déclare une source de donnée sur un tel JSON, l'autocomplétion ne fonctionne pas : on voit bien le résultat s'afficher dans le widget, mais quand on valide aucune donnée n'a été enregistrée.


Fichiers


Demandes liées

Lié à w.c.s. - Development #46031: lever une erreur quand get_value_by_id() donne quelque chose avec un id différentFermé24 août 2020

Actions

Révisions associées

Révision fd9afcc5 (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

data sources: check identifiers as strings (#47974)

Révision 6f97cef2 (diff)
Ajouté par Frédéric Péters il y a plus de 3 ans

tests: extend basic item data source check with json&numeric cases (#47974)

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 3 ans

Tu veux que je regarde pour une correction ce soir ?

#3

Mis à jour par Thomas Noël il y a plus de 3 ans

Frédéric Péters a écrit :

Tu veux que je regarde pour une correction ce soir ?

Un oeil de plus est bienvenu, je suis un peu à la ramasse sur les affaires jsonp/select2 :/

#4

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Assigné à mis à Frédéric Péters
#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

Voilà, il y a dans la branche un premier commit qui ajoute des tests sans la partie id/q, où ça fonctionnait; puis un commit avec la correction qui ajoute aussi le test avec id/q.

#7

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#8

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Lié à Development #46031: lever une erreur quand get_value_by_id() donne quelque chose avec un id différent ajouté
#9

Mis à jour par Lauréline Guérin il y a plus de 3 ans

(introduit dans #46031)

#10

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit fd9afcc57c8b976ff9ea0794764ec1a90de60a0b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Oct 22 18:11:10 2020 +0200

    data sources: check identifiers as strings (#47974)

Un grand merci, Frédéric.

#11

Mis à jour par Frédéric Péters il y a plus de 3 ans

(je ne sabote pas le build mais je pousserai le premier commit qui ajoutait des tests, qui était dans la branche mais n'a pas repris, après).

#12

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#13

Mis à jour par Thomas Noël il y a plus de 3 ans

Frédéric Péters a écrit :

(je ne sabote pas le build mais je pousserai le premier commit qui ajoutait des tests, qui était dans la branche mais n'a pas repris, après).

Manque d'attention de ma part... c'était pas ma soirée.

Formats disponibles : Atom PDF