Projet

Général

Profil

Bug #58867

toulouse_smart: accepter un payload sans block de champ pour les interventions sans propriété

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:
22 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Lorsqu'un type d'intervention ne déclare aucune propriété (ou champs),
on ne génère pas de bloc de champ pour ce type d'intervention.

Il faut que le connecteur accepte un payload sans bloc de champ pour ces interventions et qu'il passe un contenu vide à Smart.

Fichiers

0001-toulouse_smart-accept-empty-block-field-content-on-c.patch (8,8 ko) 0001-toulouse_smart-accept-empty-block-field-content-on-c.patch Nicolas Roche, 22 novembre 2021 18:45
0001-toulouse_smart-accept-empty-block-field-content-on-c.patch (7,99 ko) 0001-toulouse_smart-accept-empty-block-field-content-on-c.patch Nicolas Roche, 23 novembre 2021 08:50

Révisions associées

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

toulouse_smart: accept empty block-field content on create intervention (#58867)

Historique

#2

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

  • Description mis à jour (diff)
#3

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

#4

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

(j'ai juste ajouté un if, mais du coup j'ai du décaler son contenu à droite)

#5

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

  • Statut changé de Solution proposée à En cours

Je recopie ici une proposition qui me parait plus simple et suffisante :

        try:
            block = post_data['fields']['%s_raw' % wcs_block_varname][0]
        except:
            block = {}

Sauf si une propriété requise est absente, l'absence du bloc ou de certaines propriétés on s'en fout un peu.

#6

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

  • Statut changé de En cours à Solution proposée

si une propriété requise est absente, l'absence du bloc on s'en fout un peu.

oui, le message d'erreur mentionnera également le bloc.

#8

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit d8685c577557f9ff76102246c5e0ba5c06f2180a
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Mon Nov 22 18:33:09 2021 +0100

    toulouse_smart: accept empty block-field content on create intervention (#58867)
#11

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