backoffice: add popup to cleanup logged errors (#40821) #1024
lguerin
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/40821-logged-errors-cleanup"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
a524f2e79bto72f60af5e8WIP: backoffice: add popup to cleanup logged errors (#40821)to backoffice: add popup to cleanup logged errors (#40821)@ -209,3 +211,3 @@self.workflow_id = workflow_iddef _q_index(self):def _q_access(self):Ajout de la classe AccessControlled, le contrôle d'acccès qui était posé en haut de _q_index est mis dans la nouvelle méthode _q_access, pour être effectif également pour la nouvelle vue.
@ -243,0 +256,4 @@form.add(CheckboxesWidget,'types',title=_('Error types'),Un champ pour choisir les types d'erreur.
@ -243,0 +265,4 @@DateWidget,'latest_occurence',title=_('Latest occurence'),value=datetime.date.today() - datetime.timedelta(days=180),Un autre pour choisir la date max pour la dernière occurence, valeur par défaut à à peu près 6 mois, c'est assez arbitraire.
@ -243,0 +281,4 @@if 'carddef' in form.get_widget('types').parse():type_criterias.append(Equal('formdef_class', 'CardDef'))if 'others' in form.get_widget('types').parse():type_criterias.append(Null('formdef_class'))Possible d'ajouter un test avec aucune des cases présente cochée ? Juste pour vérifier que le form sera en erreur (parce que champ required), et qu'on ne passera pas dans un cas où il n'y a pas de critère sur formdef_class, ce qui supprimerait tout.
Voilà j'ai ajouté ce test, ainsi que quelques lignes de commentaire dans le test pour expliciter les différentes intentions.
72f60af5e8tode2bfd07ea