[Bridge/Doctrine] remove workarounds from the past

nicolas-grekas · nicolas-grekas · commit f8d6d07b5d97 · 2018-10-31T17:56:39.000+01:00
diff --git a/src/Symfony/Bridge/Doctrine/Messenger/DoctrineTransactionMiddleware.php b/src/Symfony/Bridge/Doctrine/Messenger/DoctrineTransactionMiddleware.php
@@ -27,7 +27,7 @@ class DoctrineTransactionMiddleware implements MiddlewareInterface
     private $managerRegistry;
     private $entityManagerName;
 
-    public function __construct(ManagerRegistry $managerRegistry, ?string $entityManagerName)
+    public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName = null)
     {
         $this->managerRegistry = $managerRegistry;
         $this->entityManagerName = $entityManagerName;
diff --git a/src/Symfony/Bridge/Doctrine/Messenger/DoctrineTransactionMiddlewareFactory.php b/src/Symfony/Bridge/Doctrine/Messenger/DoctrineTransactionMiddlewareFactory.php
diff --git a/src/Symfony/Bridge/Doctrine/Tests/Messenger/DoctrineTransactionMiddlewareTest.php b/src/Symfony/Bridge/Doctrine/Tests/Messenger/DoctrineTransactionMiddlewareTest.php
@@ -34,7 +34,7 @@ public function setUp()
         $managerRegistry = $this->createMock(ManagerRegistry::class);
         $managerRegistry->method('getManager')->willReturn($this->entityManager);
 
-        $this->middleware = new DoctrineTransactionMiddleware($managerRegistry, null);
+        $this->middleware = new DoctrineTransactionMiddleware($managerRegistry);
     }
 
     public function testMiddlewareWrapsInTransactionAndFlushes()

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ class DoctrineTransactionMiddleware implements MiddlewareInterface`
`27`	`27`	`private $managerRegistry;`
`28`	`28`	`private $entityManagerName;`
`29`	`29`
`30`		`- public function __construct(ManagerRegistry $managerRegistry, ?string $entityManagerName)`
	`30`	`+ public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName = null)`
`31`	`31`	`{`
`32`	`32`	`$this->managerRegistry = $managerRegistry;`
`33`	`33`	`$this->entityManagerName = $entityManagerName;`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function setUp()`
`34`	`34`	`$managerRegistry = $this->createMock(ManagerRegistry::class);`
`35`	`35`	`$managerRegistry->method('getManager')->willReturn($this->entityManager);`
`36`	`36`
`37`		`- $this->middleware = new DoctrineTransactionMiddleware($managerRegistry, null);`
	`37`	`+ $this->middleware = new DoctrineTransactionMiddleware($managerRegistry);`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`public function testMiddlewareWrapsInTransactionAndFlushes()`