Simplify some code with null coalesce operator

javiereguiluz · derrabus · commit bd7bcf556491 · 2021-07-18T18:24:15.000+02:00
diff --git a/ContainerAwareEventManager.php b/ContainerAwareEventManager.php
@@ -54,7 +54,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
@@ -163,7 +163,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
`@@ -54,7 +54,7 @@ public function dispatchEvent($eventName, EventArgs $eventArgs = null)`
`54`	`54`	`return;`
`55`	`55`	`}`
`56`	`56`
`57`		`- $eventArgs = null === $eventArgs ? EventArgs::getEmptyInstance() : $eventArgs;`
	`57`	`+ $eventArgs = $eventArgs ?? EventArgs::getEmptyInstance();`
`58`	`58`
`59`	`59`	`if (!isset($this->initialized[$eventName])) {`
`60`	`60`	`$this->initializeListeners($eventName);`
Original file line number	Diff line number	Diff line change
`@@ -163,7 +163,7 @@ public function validate($entity, Constraint $constraint)`
`163`	`163`	`return;`
`164`	`164`	`}`
`165`	`165`
`166`		`- $errorPath = null !== $constraint->errorPath ? $constraint->errorPath : $fields[0];`
	`166`	`+ $errorPath = $constraint->errorPath ?? $fields[0];`
`167`	`167`	`$invalidValue = $criteria[$errorPath] ?? $criteria[$fields[0]];`
`168`	`168`
`169`	`169`	`$this->context->buildViolation($constraint->message)`