Projet

Général

Profil

Bug #56630

toulouse_smart: passer les paramètres facultatifs à la création

Ajouté par Nicolas Roche il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Nicolas Roche
Version cible:
-
Début:
03 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

(oubli)

Fichiers

0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch (3,07 ko) 0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch Nicolas Roche, 03 septembre 2021 15:49
0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch (5,28 ko) 0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch Nicolas Roche, 03 septembre 2021 17:47
0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch (5,87 ko) 0001-toulouse_smart-pass-optionnal-boolean-parameters-to-.patch Nicolas Roche, 03 septembre 2021 18:51

Révisions associées

Révision f39941dd (diff)
Ajouté par Nicolas Roche il y a plus de 2 ans

toulouse_smart: pass optionnal boolean parameters to create endpoint (#56630)

Historique

#1

Mis à jour par Nicolas Roche il y a plus de 2 ans

Ce n'est évident de passer un booléen via l'action WS de w.c.s mais j'ai quand même laissé ce type dans le schéma JSON de validation parce que j'ai vu que ça avait été fait ailleurs (en passant par une case à cocher : {{ form_var_morning_raw }}) :
https://demarches-isere.test.entrouvert.org/backoffice/workflows/68/status/1/items/1/
https://demarches-isere.test.entrouvert.org/backoffice/forms/116/fields/4/

#2

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

Ne connaissant pas toulouse_smart je ne comprends ni le ticket ni le patch, mais à toutes fins utiles, pour aider wcs et la gestion des booléens, on a inventé ce type dans le connecteur Axel :

    ....
    'oneOf': [
        {'type': 'boolean'},
        {
            'type': 'string',
            'pattern': '[Oo][Uu][Ii]|[Nn][Oo][Nn]|[Tt][Rr][Uu][Ee]|[Ff][Aa][Ll][Ss][Ee]|1|0',
        },
    ]

faut faire un peu de traitement derrière en cas de string, mais bon bref.

#4

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

"oneOf"

#5

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Le plus simple c'est d'en faire une constante 

BOOLEAN_TYPE = { 'oneOf': ... }

#7

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

J'aurai plutôt vu 

BOOLEAN_TYPE = {'oneOf': BOOLEAN_TYPES}
...

    'type': BOOLEAN_TYPE,

qui est ce qui est fait ailleurs mais go.

#8

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)

J'aurai plutôt vu ...

oui c'est ce que j'ai essayé en premier, mais ça ne passait pas.

commit f39941dd935911c8f94cfefb612d9c6bf83918f1
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Sep 3 15:21:27 2021 +0200

    toulouse_smart: pass optionnal boolean parameters to create endpoint (#56630)
#9

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

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

Formats disponibles : Atom PDF