Projet

Général

Profil

Development #32472

Planitec : le code de tarif doit pouvoir être passé dynamiquement

Ajouté par Emmanuel Cazenave il y a environ 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
18 avril 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Le code tarifaire doit pouvoir changer en fonction d'un champ de formulaire (Vénissian/non Vénissian) et donc être passé dynamiquement au endpoint getreservationprice (qui crée en sous main un utilisateur Planitec, sur lequel le code tarifaire est passé).

#32216#note-6


Fichiers

Révisions associées

Révision e8446c71 (diff)
Ajouté par Emmanuel Cazenave il y a presque 5 ans

planitec: change price code dynamically (#32472)

Historique

#2

Mis à jour par Emmanuel Cazenave il y a presque 5 ans

Testé avec la dernière version de planitec (#32631#note-7), ça fonctionne.

#3

Mis à jour par Emmanuel Cazenave il y a presque 5 ans

#4

Mis à jour par Thomas Noël il y a presque 5 ans

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

Ma nature pénible te demanderait bien de renommer updateorcreate en update_or_create mais ça va aller.

Et sans savoir si c'est pertinent, je me demande si la partie

                # Update planitec user
                pairing.price_code = dyn_price_code
                pairing.save()

ne devrait pas être faite uniquement si l'appel à Planitec a réussi (donc décaler cela en dessous du if data.get('modificationStatus') != 'OK':)

Rien d'autre à dire donc je valide quand même ta proposition, à toi de voir si ce que je dis au dessus est (f)utile.

#5

Mis à jour par Emmanuel Cazenave il y a presque 5 ans

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

Thomas Noël a écrit :

Ma nature pénible te demanderait bien de renommer updateorcreate en update_or_create mais ça va aller.

Poussé avec ce changement.

commit e8446c718340a87f3ebdde5431fbb8e3a6155cc5
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date:   Thu Apr 25 15:23:36 2019 +0200

    planitec: change price code dynamically (#32472)

[...]
ne devrait pas être faite uniquement si l'appel à Planitec a réussi (donc décaler cela en dessous du if data.get('modificationStatus') != 'OK':)

Laissé tel quel parce que le tout est encapsulé dans une transaction.

#6

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

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

Formats disponibles : Atom PDF