Produits Entr'ouvert » w.c.s.

Ok, ça doit plutôt être du bricolage d'exposer ça sous /api/.../, on va laisser cette partie pour plus tard. Cela étant quand même, j'incluerais l'url du formdata dans le dictionnaire, plutôt que de demander à l'appelant de construire par lui-même cette URL.

Je viens de valider sur un début de test unitaire et ça ne va pas passer; de manière globale l'accès à /backoffice/ est bloqué ainsi :

        user = get_request().user
        if not user and get_publisher().user_class.count() > 0:
            raise errors.AccessUnauthorizedError(
                    public_msg = _('Access to backoffice is restricted to authorized persons only. '\
                                   'Please login.'))

On sait qu'on n'a pas envie de mélanger get_request().user et le get_user_from_api_query_string. À mon sens on se trouve ici finalement obligé d'aller vers le /api/; en y mettant un ApiFormPage héritant du FormPage de backoffice/root.py, avec un _q_export n'exposant que le json. On garderait donc le patch comme il est, ce sera toujours utile de pouvoir faire un /json de test depuis le backoffice; mais en parallèle on crée le ApiFormPage(FormPage) qu'on place sous /api/.

Ok. Je compte pousser le patch avec un message de commit différent :

    backoffice/listings: add minimal JSON export (#6642)

    also move JSONEncoder to qommon.misc

Ok ?

(poussé ainsi)

Vrac.

Si on a un peu de temps pour réfléchir à la structure d'URL, allons-y; j'éviterais le '/json', pourquoi pas /api/forms/<urlname>/list ?

Par ailleurs, les URL données pour les formdata sont des /backoffice/... qui ne seront pas accessibles (ajouter un paramètre api=False au get_url() des formdata ?) (ou simplement retirer le backoffice=True pour le moment ?).

Dans le init de FormPage, il y a une vérification de get_request().user, il me semble qu'il faut donc y mettre un init propre, ou factoriser un peu l'existant.

Et dans le _q_lookup de ApiFormsDirectory, je ne suis pas sûr du "or get_request().user", je limiterais aux utilisateurs spécifiques API (j'entends bien l'avantage pour le debug, donc peut-être prendre en considération get_request().user si celui-ci est admin ?).

Ça m'a l'air ok (j'aurais mis le forms = ... au-dessus des méthodes de la classe); seul truc, ce n'était pas l'occasion d'écrire pour toi d'écrire des tests ?