minor #5828 move the getEntityManager, only get it if needed (OskarStark)

wouterj · wouterj · commit a39204125129 · 2015-11-25T15:23:14.000+01:00
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes #5828). Discussion ---------- move the getEntityManager, only get it if needed if this will be merged, please be sure, that this is the correct branch! Thank you Commits ------- 1498140 move the getEntityManager, only get it if needed
diff --git a/cookbook/doctrine/event_listeners_subscribers.rst b/cookbook/doctrine/event_listeners_subscribers.rst
@@ -135,10 +135,10 @@ a ``postPersist`` method, which will be called when the event is dispatched::
         public function postPersist(LifecycleEventArgs $args)
         {
             $entity = $args->getEntity();
-            $entityManager = $args->getEntityManager();
 
             // perhaps you only want to act on some "Product" entity
             if ($entity instanceof Product) {
+                $entityManager = $args->getEntityManager();
                 // ... do something with the Product
             }
         }
@@ -191,10 +191,10 @@ interface and have an event method for each event it subscribes to::
         public function index(LifecycleEventArgs $args)
         {
             $entity = $args->getEntity();
-            $entityManager = $args->getEntityManager();
 
             // perhaps you only want to act on some "Product" entity
             if ($entity instanceof Product) {
+                $entityManager = $args->getEntityManager();
                 // ... do something with the Product
             }
         }

Original file line number	Diff line number	Diff line change
@@ -135,10 +135,10 @@ a ``postPersist`` method, which will be called when the event is dispatched::
`135`	`135`	`public function postPersist(LifecycleEventArgs $args)`
`136`	`136`	`{`
`137`	`137`	`$entity = $args->getEntity();`
`138`		`- $entityManager = $args->getEntityManager();`
`139`	`138`
`140`	`139`	`// perhaps you only want to act on some "Product" entity`
`141`	`140`	`if ($entity instanceof Product) {`
	`141`	`+ $entityManager = $args->getEntityManager();`
`142`	`142`	`// ... do something with the Product`
`143`	`143`	`}`
`144`	`144`	`}`
`@@ -191,10 +191,10 @@ interface and have an event method for each event it subscribes to::`
`191`	`191`	`public function index(LifecycleEventArgs $args)`
`192`	`192`	`{`
`193`	`193`	`$entity = $args->getEntity();`
`194`		`- $entityManager = $args->getEntityManager();`
`195`	`194`
`196`	`195`	`// perhaps you only want to act on some "Product" entity`
`197`	`196`	`if ($entity instanceof Product) {`
	`197`	`+ $entityManager = $args->getEntityManager();`
`198`	`198`	`// ... do something with the Product`
`199`	`199`	`}`
`200`	`200`	`}`