Going back to the Caution alternative.

juanmf · weaverryan · commit d920148e1525 · 2014-03-27T11:24:13.000-04:00
diff --git a/cookbook/doctrine/file_uploads.rst b/cookbook/doctrine/file_uploads.rst
@@ -410,22 +410,13 @@ Next, refactor the ``Document`` class to take advantage of these callbacks::
         }
     }
 
-When using a Doctrine event listener or subscriber, when you make changes 
-to your entity, the preUpdate() callback must have an extra line of code to 
-tell Doctrine about the change::
-
-    public function preUpdate(PreUpdateEventArgs $args)
-    {
-        $entity = $args->getEntity();
-        // do all the file uploading logic
-        // ...
-        $entity->setFilename($newFilename);
-        $args->setNewValue('filename', $newFilename);
-    }    
-    
-For full reference on preUpdate event restrictions, see `preUpdate`_ in the 
-Doctrine Events documentation.
+.. caution::
 
+   If changes to your entity are handled by a Doctrine event listener or event 
+   subscriber, the ``preUpdate()`` callback must notify Doctrine about the changes 
+   being done.
+   For full reference on preUpdate event restrictions, see `preUpdate`_ in the 
+   Doctrine Events documentation.
 
 The class now does everything you need: it generates a unique filename before
 persisting, moves the file after persisting, and removes the file if the
