From d7511a1d162e9499c9f9a4cd82c864cf83432941 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Pineau?= Date: Thu, 3 Oct 2019 13:46:03 +0200 Subject: [PATCH] [Workflow] Removed note about flushing in `entered` event This sentence is confusing and lead to bad practices: * It's not a good idea to flush in the model layer * It can lead to many calls to `flush()` method if the user uses a workflow (with many `to` places after a transition) * It's not possible to "preview" what a transition do, because everything is flushed in the DB --- workflow.rst | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/workflow.rst b/workflow.rst index c55d4fcc269..5f53f2261c2 100644 --- a/workflow.rst +++ b/workflow.rst @@ -260,8 +260,7 @@ order: * ``workflow.[workflow name].enter.[place name]`` ``workflow.entered`` - The subject has entered in the places and the marking is updated (making it a good - place to flush data in Doctrine). + The subject has entered in the places and the marking is updated. The three events being dispatched are: