Produits Entr'ouvert » w.c.s.

(la branche associée contient également #31539 et #19271 qui viennent avant ce patch)

Ce patch crée un /api/autocomplete/<token> qui est fait proxy vers l'URL de la source de données, en signant l'appel si possible. Le <token> est associé la session (stockés dans data_source_query_url_tokens), ça limite l'accès à la session en cours et ça permet de le faire varier dynamiquement (en cas de source type https://.../rues?commune={{form_var_commune}} avec form_var_commune sur la même page).

Ensuite, c'est d'une part le code jsonp existant pour le côté javascript (en passant il n'y avait pas de tests sur cette partie jsonp et c'est maintenant le cas) + du nouveau code pour aller chercher les données structurées associées au choix en interrogeant url?id=xxx. (le nom des paramètres ?q et ?id se paramètrent dans la source de données).

Rien de bien particulier sinon.

Pas simple à relire, pas simple à écrire certainement.

Je découperai une première partie avec l'introduction de get_object(), type(), get_jsonp_url(), can_jsonp()(qui pourrait être inutile en utilisant juste get_jsonp_url() je pense) et des tests jsonp pures et je modifierai à l'occasion dans fields.py les bouts jouent sur real_data_source, je pense que ça rendrait la lecture des modifications pour la conversion JSON -> JSONP plus claire.

Je suis une peu perturbé par les histoires de jsonp_display_values qui sont difficile à suivre ce qui rend les choses compliquées, j'espère qu'on simplifiera plus tard, voir qu'on aura plus besoin du tout du code JSONP pure (j'ai conscience que ça ne concerne que le cas JSONP pure mais comme c'est au milieu du reste).

Je ne suis pas sûr de comprendre pourquoi on fait ça :

            with get_publisher().substitutions.temporary_feed(
                    transient_formdata, force_mode='lazy'):
                data = self.formdef.get_data(form)

Au niveau du nouveau test JSON, il me semble que le champ caché f0_display ne sert plus dans ce cas puisque les données sont résolues après soumission depuis la source, alors peut-être vaut-il mieux tester que cela fonctionne sans ? En plus dans le test JSONP il y a aussi un jeu sur l'ordre du champ qui n'est pas présent ici.

Des bouts de code qui pourraient être factorisés :

        unsigned_url = url
        try:
            signature_key, orig = get_secret_and_orig(url)
        except MissingSecret:
            pass
        else:
            url += '&orig=%s' % orig
            url = sign_url(url, signature_key)

Le système de jeton est ok mais pas mal d'implicite dans le fait que l'URL se termine par le paramètre de query incomplet dans get_jsonp_url() si c'est du JSON+autocomplete mais une URL simple si c'est du JSONP (ce n'est donc pas du tout le même genre d'URL qui est retourné), pour éviter un stockage dans session supplémentaire on pourrait aussi utiliser django.core.signing avec un dico {'url':..., 'session_id_hash': sha256(session_id)}.

Globalement le code ne m'inquiète pas mais c'est pas facile à suivre.

(branche à jour avec tout ça)

Ok.

commit 3bf61b16b3e0fe260157020d048ae54edd2036a7
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Apr 4 14:45:00 2019 +0200

    forms: add support for live change of JSON data source (#31492)

commit 065247ae13110980e0080652fceb139b0a38f75c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Apr 4 15:22:50 2019 +0200

    forms: extend json autocomplete support for be asynchronous if possible (#31492)

commit 45a374ae50cad7d7f49cdac8d4b248a696351b26
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Apr 4 15:22:24 2019 +0200

    api utils: add function to sign an URL if orig is known (#31492)

commit 5351db8a2ed204779b4ca93a1f36bb60c351ef33
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Apr 4 14:43:42 2019 +0200

    misc: refactor and test jsonp data source (#31492)