Development #69972
phonecalls: sur un callstart d'un appel déjà existant, créer un nouvel appel
0%
Description
On voit des PBX qui ne savent pas envoyer des callstop. On se retrouve avec des appels qui durent tous max_call_duration (2 heures).
Et lorsqu'un même numéro rappelle avant cette expiration, on pense que c'est un appel en cours et on ne fait quasi rien :
def call_start(self, request, callee, caller, redirect=None, newtab=None, **kwargs): existing_call = Call.objects.filter( resource=self, callee=callee, caller=caller, end_timestamp=None ).last() if existing_call: existing_call.details = kwargs existing_call.save() response = {'data': existing_call.json()}
Conséquence, c'est comme s'il n'y avait pas de nouvel appel, rien n'est rafraîchi sur le portail agent. C'est vraiment visible dans mode test quand on essaye avec un ou deux numéros.
On devrait soit couper toutes les connexion existantes et en créer une nouvelle ou, plus simplement, mettre à jour le "start_timestamp" de l'appel de le plus récent.
Fichiers
Révisions associées
phonecalls: do not consider existing calls on call start (#69972)
Historique
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Plutôt que de changer start_date, ce qui perdra l'existence de l'appel précédent dans l'historique je proposerai plutôt de le fermer et le de réouvrir (et de ne pas fermer tous les autres appels, on ne sait jamais s'ils ont du double appel :) ).
Mis à jour par Thomas Noël il y a plus d'un an
Benjamin Dauvergne a écrit :
Plutôt que de changer start_date, ce qui perdra l'existence de l'appel précédent dans l'historique je proposerai plutôt de le fermer et le de réouvrir (et de ne pas fermer tous les autres appels, on ne sait jamais s'ils ont du double appel :) ).
Oui, bonne idée, et après tout c'est plus en phase avec la réalité.
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-phonecalls-do-not-consider-existing-calls-on-call-st.patch 0001-phonecalls-do-not-consider-existing-calls-on-call-st.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà une version simplifiée qui ajoute un nouvel appel quand il reçoit un callstart.
Je pense inutile toute intelligence, je n'imagine pas un PBX nous envoyer plusieurs callstart pour un même appel.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Thomas Noël a écrit :
Je pense inutile toute intelligence, je n'imagine pas un PBX nous envoyer plusieurs callstart pour un même appel.
Tu as toujours été un grand optimiste.
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 1d8bcee7065fc850f8147a19e31d1cf4bc5310f2 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Thu Oct 6 18:40:30 2022 +0200 phonecalls: do not consider existing calls on call start (#69972)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Thomas Noël il y a plus d'un an
- Sujet changé de phonecalls: sur un callstart d'un appel déjà existant, mettre à jour sa date de début à phonecalls: sur un callstart d'un appel déjà existant, créer un nouvel appel
phonecalls: do not consider existing calls on call start (#69972)