Produits Entr'ouvert » w.c.s.

Voilà le patch, mais je ne mets pas "rustine proposée" encore, parce que le relecteur avisé notera que :

• on ne loggue pas le contexte complet du calcul de la condition. (alors qu'actuellement on a la trace complète, mais "hors sol", liée ni à un formulaire ni à une action)

On pourrait ajouter cette possibilité, mais le lien vers l'inspecteur de la demande qui a planté, et l'action en cause, ça me semble suffisant. (Peut-être pourrait-on plutôt logguer l'id de la dernière demande qui a planté, histoire de regarder l'inspecteur d'une demande qui n'a pas encore trop bougé ?)

• dans les tests, ce moment assert logged_error.occurences_count > 1 # should be 2... == 12 with pickle, 4 with sql

On voit que la modif d'un workflow et le recalcul des droits qui l'accompagne lance plusieurs fois les calculs des conditions. Le code qui fait cette partie me semble cependant déjà pas mal "optimisé", du moins je n'y ai pas encore vu de soucis.

Voici une version qui simplifie le message affiché, qui se résume à "Echec lors du calcul d'une condition (erreur Exception)". Quand on clique dessus, quelques détails, la condition en cause, et le message plus complet de l'exception (et bien sûr, comme actuellement, le premier formulaire impacté, le form, le wf, statut, l'action, etc.).

Sur « on calcule peut-être "un peu trop" les conditions » : je n'ai pas réussi à bien voir le soucis encore ; serait plutôt l'objet d'un patch séparé.

Bonnes idées, dont voici une implémentation possible.

Voilà, avec l'interdiff :

diff --git a/wcs/admin/logged_errors.py b/wcs/admin/logged_errors.py
index 542ad5ec..106da872 100644
--- a/wcs/admin/logged_errors.py
+++ b/wcs/admin/logged_errors.py
@@ -78,11 +78,11 @@ class LoggedErrorDirectory(Directory):
                 'template': N_('Template'),
                 'text': N_('Text'),
             }.get(self.error.expression_type, N_('Unknown'))
-            r += htmltext('  <li>%s <pre>%s</pre></li>') % (
+            r += htmltext('  <li>%s %s</li>') % (
                     _('%s:') % expression_title, self.error.expression)

         if self.error.exception_class or self.error.exception_message:
-            r = htmltext('  <li>%s 
%s: %s
</li>') % (,
            r += htmltext('  <li>%s %s: %s</li>') % (('Error message:'),
                                                                 self.error.exception_class,
                                                                 self.error.exception_message)

	
@ -175,9 +175,9 @ class LoggedErrorsDirectory(Directory):
                 '%(count)d error', '%(count)d errors', len(errors)) % {'count': len(errors)}
         r = htmltext('<ul>')
         for error in errors[:3]:
-            r += htmltext('<li><a href="logged-errors/%s/">%s</a>') % (error.id, error.summary)
            r = htmltext('<li><a href="logged-errors/%s/">%s</a> ') % (error.id, error.summary)
             if error.exception_class or error.exception_message:
-                r += htmltext(s (s)')) % {
                r = htmltext(('error %(class)s ((message)s)')) % {
                     'class': error.exception_class,
                     'message': error.exception_message,
                 }
@ -201,9 +201,9 @ class LoggedErrorsDirectory(Directory):
         r += htmltext('<h2>%s</h2>') % 
         r += htmltext('<ul class="biglist">')
         for error in self.get_errors(formdef_id=self.formdef_id, workflow_id=self.workflow_id):
-            r += htmltext('<li><strong><a href="%s/">%s</a></strong>') % (error.id, error.summary)
            r = htmltext('<li><strong><a href="%s/">%s</a></strong> ') % (error.id, error.summary)
             if error.exception_class or error.exception_message:
-                r += htmltext((' error s ((message)s)')) % {
                r += htmltext(_('error s ((message)s)')) % {
                     'class': error.exception_class,
                     'message': error.exception_message,
                 }

À part ça, je dirais ok; je me demande juste si à un moment on ne devra pas réfléchir à enregistrer la stack, plutôt pour nous débugguer quand le reste ne suffit pas. (mais laissons ça à plus tard).

Je me disais de mon côté, enregistrer les 10 derniers (formdef, wf, formdata, status, status_item, datetime) en cause, histoire de ne pas conserver que le premier formdata en cause (qui risque d'avoir bougé sinon disparu après quelques jours) → #25086. Bien d'autres tickets à faire pour améliorer tout ça, oui.

commit 25150c2136611da6227e94ea894bf7eaf598efb8
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Tue Jun 12 16:49:12 2018 +0200

    workflows: only log condition errors as functional errors (#24472)