Merge branch '5.2' into 5.3

derrabus · derrabus · commit 9bec1a8417d5 · 2021-07-18T18:30:56.000+02:00
* 5.2:
  Simplify some code with null coalesce operator
  Don't use deprecated TestLogger class
diff --git a/ContainerAwareEventManager.php b/ContainerAwareEventManager.php
@@ -58,7 +58,7 @@ public function dispatchEvent($eventName, EventArgs $eventArgs = null)
             return;
         }
 
-        $eventArgs = null === $eventArgs ? EventArgs::getEmptyInstance() : $eventArgs;
+        $eventArgs = $eventArgs ?? EventArgs::getEmptyInstance();
 
         if (!isset($this->initialized[$eventName])) {
             $this->initializeListeners($eventName);
diff --git a/Validator/Constraints/UniqueEntityValidator.php b/Validator/Constraints/UniqueEntityValidator.php
@@ -165,7 +165,7 @@ public function validate($entity, Constraint $constraint)
             return;
         }
 
-        $errorPath = null !== $constraint->errorPath ? $constraint->errorPath : $fields[0];
+        $errorPath = $constraint->errorPath ?? $fields[0];
         $invalidValue = $criteria[$errorPath] ?? $criteria[$fields[0]];
 
         $this->context->buildViolation($constraint->message)

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ public function dispatchEvent($eventName, EventArgs $eventArgs = null)`
`58`	`58`	`return;`
`59`	`59`	`}`
`60`	`60`
`61`		`- $eventArgs = null === $eventArgs ? EventArgs::getEmptyInstance() : $eventArgs;`
	`61`	`+ $eventArgs = $eventArgs ?? EventArgs::getEmptyInstance();`
`62`	`62`
`63`	`63`	`if (!isset($this->initialized[$eventName])) {`
`64`	`64`	`$this->initializeListeners($eventName);`
Original file line number	Diff line number	Diff line change
`@@ -165,7 +165,7 @@ public function validate($entity, Constraint $constraint)`
`165`	`165`	`return;`
`166`	`166`	`}`
`167`	`167`
`168`		`- $errorPath = null !== $constraint->errorPath ? $constraint->errorPath : $fields[0];`
	`168`	`+ $errorPath = $constraint->errorPath ?? $fields[0];`
`169`	`169`	`$invalidValue = $criteria[$errorPath] ?? $criteria[$fields[0]];`
`170`	`170`
`171`	`171`	`$this->context->buildViolation($constraint->message)`