Projet

Général

Profil

Development #12416

wscall ne supporte pas le verbe DELETE

Ajouté par Benjamin Dauvergne il y a presque 8 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Benjamin Dauvergne
Version cible:
-
Début:
02 juillet 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Au passage on peut ajouter PUT aussi, ça mange pas de pain.

Fichiers

0001-implement-PUT-PATCH-and-DELETE-for-webservice-calls-.patch (8,7 ko) 0001-implement-PUT-PATCH-and-DELETE-for-webservice-calls-.patch Benjamin Dauvergne, 02 juillet 2016 18:59
0001-implement-PUT-PATCH-and-DELETE-for-webservice-calls-.patch (8,73 ko) 0001-implement-PUT-PATCH-and-DELETE-for-webservice-calls-.patch Benjamin Dauvergne, 10 mars 2017 00:19
0001-wscalls-implement-PUT-PATCH-and-DELETE-verbs-12416.patch (8,33 ko) 0001-wscalls-implement-PUT-PATCH-and-DELETE-verbs-12416.patch Frédéric Péters, 31 août 2018 09:02

Révisions associées

Révision c5f786d2 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 5 ans

wscalls: implement PUT, PATCH and DELETE verbs (#12416)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Il y a un besoin pour supprimer les booking dans chrono.

#3

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

  • Assigné à mis à Benjamin Dauvergne
#4

Mis à jour par Frédéric Péters il y a presque 8 ans

Non, Chrono n'amène pas ce besoin. (#11472).

#5

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

  • Priorité changé de Normal à Bas

Ok je met priorité basse.

#6

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

  • Priorité changé de Bas à Normal

Je remonte ce ticket car avec CUT et Zoo, je pense que ces méthodes vont devenir utiles sinon indispensables, parce qu'on utilise DjangoRestFramework dans ces applis.

#8

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

  • Statut changé de Nouveau à En cours

Sans doute à rebaser avant de le passer en "solution proposée".

#9

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

Patch rebasé et corrigé,

            [
                ('GET', _('GET')),
                ('POST', _('POST (JSON)'))
                ('PUT', _('PUT (JSON)')),

Manquait une virgule en fin de ligne du POST.

+ adaptation du mock dans test_sms_with_passerelle.

#10

Mis à jour par Paul Marillonnet il y a plus de 5 ans

Est-ce que, quitte à modifier le code dans call_webservice, on aurait pas intérêt à tout regrouper en un seul appel à _http_request ?

(préparer payload et headers quand la méthode est {PUT,PATCH,POST}, et récupérer request quand la méthode est get).
(et en plus l'implémentation http_get_page ne consiste qu'à appeler _http_request avec les mêmes arguments.)

#11

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

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

Il y a trois situations (payload (PATH, PUT, POST), cache (GET), rien (DELETE)), la situation proposée avec trois branches me va bien. Et d'ailleurs, comme c'est avant tout le code de Benjamin, je vais le valider.

#12

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit c5f786d22269a0632e4368f4c74290a250bd66bf
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Jul 2 18:44:13 2016 +0200

    wscalls: implement PUT, PATCH and DELETE verbs (#12416)
#13

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

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

Formats disponibles : Atom PDF