8000 minor #12413 [Workflow] Removed note about flushing in `entered` even… · symfony/symfony-docs@659546e · GitHub
[go: up one dir, main page]
Skip to content

Commit 659546e

Browse files
javiereguiluzjaviereguiluz
committed
minor #12413 [Workflow] Removed note about flushing in entered event (lyrixx)
This PR was merged into the 3.4 branch. Discussion ---------- [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 Commits ------- d7511a1 [Workflow] Removed note about flushing in `entered` event
2 parents b3d7f8b + d7511a1 commit 659546e

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

‎workflow.rst

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,8 +260,7 @@ order:
260260
* ``workflow.[workflow name].enter.[place name]``
261261

262262
``workflow.entered``
263-
The subject has entered in the places and the marking is updated (making it a good
264-
place to flush data in Doctrine).
263+
The subject has entered in the places and the marking is updated.
265264

266265
The three events being dispatched are:
267266

0 commit comments

Comments
 (0)
0