Projet

Général

Profil

Bug #19503

manager : ne pas se mettre en erreur quand le .render() d'une cellule échoue

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
17 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Pour la découverte des placeholders on passe par un rendu des cellules mais le résultat c'est aujourd'hui qu'une erreur de programmation/template/whatever sur la cellule, plutôt que "juste" cracher en frontoffice (et la plupart du temps lors d'un rendu asynchrone qui ne cassera pas toute la page), ça crashe en backoffice, directement, et il n'y a rien à faire même pas possibilité de virer la cellule la main est perdu.


Fichiers

Révisions associées

Révision 8bf25cfa (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

manager: don't crash on buggy cells (#19503)

Historique

#1

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

"Grâce" à un crash de la première version de #19368 :

File "/home/fred/src/eo/combo/combo/apps/calendar/utils.py" in get_slots
  163.         start = self.get_minimum_slot()
File "/home/fred/src/eo/combo/combo/apps/calendar/utils.py" in get_minimum_slot
  198.         return min([day.get_minimum_slot().date_time.time() for day in self.days])

Exception Value: min() arg is an empty sequence
#2

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

On veut même pas logguer un truc en debug ? Allez comme c'est juste l'étape de découverte des placeholders, on va dire qu'on s'en fiche... Ack.

#3

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

  • Statut changé de En cours à Résolu (à déployer)

C'est vrai qu'ajouter du log ça pourrait être utile, mais zappé.

commit 8bf25cfabd1f2eb65e2822a7d60cd0ed6473d2bd
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Oct 17 20:12:56 2017 +0200

    manager: don't crash on buggy cells (#19503)
#4

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