Development #11077
agoraplus: webservice de modification des coordonnées téléphoniques
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier
0001-agoraplus-phones-numbers-update-endpoint-11077.patchsupprimé
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
Patch refactorisé
Mis à jour par Thomas Noël il y a presque 8 ans
C'est pas facile à lire, parce que adult est parfois celui de wcs, parfois celui du pivot, parfois celui d'agora...
Le get_adult par exemple renvoie agora+ et pas du pivot (non normalisé). Pour moi, c'est pas ok.
Ensuite, attention, wcs peut envoyer des id qui sont des sas_xx dans le cas de familles qui ont été créées via le sas (même après leur injection dans agora+)
Enfin, je pense qu'il faut éviter d'avoir des fonction avec des arugments du genre "adult" alors que tu n'utilise que "adult[id]" : autant directement demander un argument adult_id ; ça clarifie un peu.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
Ok, avec la récuperation de l'id original de l'adulte à partir du sas_id, si défini.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
Un peu de refactoring en deplaçant la logique de récuperatin de l'id de l'adulte dans le modèle.
Mis à jour par Frédéric Péters il y a presque 8 ans
def get_agoraplus_adult_id(login, adult_id):
Manque le self. Ce code n'a jamais été exécuté.
Mis à jour par Thomas Noël il y a presque 8 ans
Le « family = self.object.get_agoraplus_family(login=self.login) » ne devrait pas non plus être dans la vue, mais dans le modèle. Dans la vue tu attrapes juste login et les numéro de phone nouveaux, et tu envoies à update_phones...
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
Dans l'idée de ne pas raiser un ObjectDoesNotExist(qui sera propagé jusqu'à jsonresponse, cf. #11058) quand il ne s'agit pas d'un modèle, je l'intercepte dans la vue et renvoie une 404.
Mis à jour par Thomas Noël il y a presque 8 ans
pour finir, ajouter « if not agoraplus_adult_id: raise ObjectDoesNotExist('adult not in Agora+') »
Et au passage, faire du i18n ces messages d'exception, car ils seront vu dans w.c.s.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
On vient de discuter avec Thomas à l'instant: on laisse tomber la i18n des chaînes des exceptions pour l'instant car jsonresponse crache lors de la tentative de serializer un objet "ugettext_lazy"
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
Yep, je le verrais dans jsonresponse
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Fichier 0001-agoraplus-phones-numbers-update-endpoint-11077.patch 0001-agoraplus-phones-numbers-update-endpoint-11077.patch ajouté
Avec les textes des exceptions internationalisés.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
commit b5d71725558166adc5d44db99bd7ddeec8beac96
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Thu May 26 11:40:16 2016 +0200
agoraplus: phones numbers update endpoint (#11077)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
Mis à jour par Thomas Noël il y a presque 8 ans
- Duplique Development #10491: agoraplus: modif téléphone ajouté
agoraplus: phones numbers update endpoint (#11077)