Produits Entr'ouvert » w.c.s.

Si un statut d'un workflow a été supprimé alors que des demandes ont ce statut enregistré comme "marqueur précédent", on a un KeyError dans pop_previous_marked_status :

    def pop_previous_marked_status(self):
        if not self.workflow_data or not '_markers_stack' in self.workflow_data:
            return None
        try:
            marker_data = self.workflow_data['_markers_stack'].pop()
            status_id = marker_data['status_id']
        except IndexError:
            return None
        return self.formdef.workflow.get_status(status_id)       <---- KeyError, le statut n'existe plus

Un petit test qui montre le soucis :

diff --git a/tests/test_backoffice_pages.py b/tests/test_backoffice_pages.py
index d4921da6..a8eb586a 100644
--- a/tests/test_backoffice_pages.py
+++ b/tests/test_backoffice_pages.py
@@ -5247,6 +5247,25 @@ def test_workflow_jump_previous_auto(pub):
     assert formdata.status == 'wf-south'
     assert formdata.workflow_data['_markers_stack'] == [{'status_id': 'north'}]

+    # no marker (workflow inconsistency)
+    formdata.workflow_data['_markers_stack'] = []
+    formdata.store()
+    resp = app.get('/backoffice/management/form-title/%s/' % formdata.id)
+    resp = resp.form.submit('button_back')
+    resp = resp.follow()
+    formdata = formdef.data_class().get(formdata.id)
+    assert formdata.status == 'wf-south'
+    assert not formdata.workflow_data['_markers_stack']
+
+    # unknown marker (workflow inconsistency)
+    formdata.workflow_data['_markers_stack'] = [{'status_id': 'unknown_status'}]
+    formdata.store()
+    resp = app.get('/backoffice/management/form-title/%s/' % formdata.id)
+    resp = resp.form.submit('button_back')
+    resp = resp.follow()
+    formdata = formdef.data_class().get(formdata.id)
+    assert formdata.status == 'wf-south'
+

Le premier test sans aucun marqueur passe, mais pas le second où le statut pointé par le marqueur n'existe pas.

(À noter qu'à ma connaissance ça n'est jamais arrivé en vrai, j'ai juste repéré le truc suite à la relecture de #42854)