Apply suggestions from code review

Co-authored-by: Claudio Zizza <859964+SenseException@users.noreply.github.com>
doctrine · Jan 10, 2023 · fe64ec2 · fe64ec2
1 parent f9a1e37
commit fe64ec2
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/lib/Doctrine/ORM/Mapping/MappingException.php b/lib/Doctrine/ORM/Mapping/MappingException.php
@@ -221,7 +221,7 @@ public static function joinColumnMustPointToMappedField(string $className, strin
     }
 
     /** @param class-string $className */
-    public static function classIsNotAValidEntityOrMappedSuperClass($className): self
+    public static function classIsNotAValidEntityOrMappedSuperClass(string $className): self
     {
         $parent = get_parent_class($className);
         if ($parent !== false) {
@@ -298,8 +298,7 @@ public static function unsupportedOptimisticLockingType(string $entity, string $
         ));
     }
 
-    /** @param string|null $path */
-    public static function fileMappingDriversRequireConfiguredDirectoryPath($path = null): self
+    public static function fileMappingDriversRequireConfiguredDirectoryPath(string|null $path = null): self
     {
         if (! empty($path)) {
             $path = '[' . $path . ']';
@@ -506,7 +505,7 @@ public static function invalidTargetEntityClass(string $targetEntity, string $so
     /** @param string[] $cascades */
     public static function invalidCascadeOption(array $cascades, string $className, string $propertyName): self
     {
-        $cascades = implode(', ', array_map(static fn (string $e) => "'" . $e . "'", $cascades));
+        $cascades = implode(', ', array_map(static fn (string $e): string => "'" . $e . "'", $cascades));
 
         return new self(sprintf(
             "You have specified invalid cascade options for %s::$%s: %s; available options: 'remove', 'persist', 'refresh', 'merge', and 'detach'",