DevKit updates

sonata-project · Dec 17, 2023 · 1df71c0 · 1df71c0
1 parent e7c1db9
commit 1df71c0
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -27,8 +27,9 @@
 HEADER;
 
 $rules = [
-    '@PHP74Migration' => true,
-    '@PHP74Migration:risky' => true,
+    '@PHP80Migration' => true,
+    '@PHP80Migration:risky' => true,
+    '@PHPUnit84Migration:risky' => true,
     '@PSR12' => true,
     '@PSR12:risky' => true,
     '@Symfony' => true,
@@ -42,6 +43,7 @@
     'multiline_whitespace_before_semicolons' => ['strategy' => 'no_multi_line'],
     'no_useless_else' => true,
     'no_useless_return' => true,
+    'no_superfluous_elseif' => true,
     'no_superfluous_phpdoc_tags' => ['allow_mixed' => true, 'remove_inheritdoc' => true],
     'nullable_type_declaration_for_default_null_value' => ['use_nullable_type_declaration' => true],
     'ordered_class_elements' => true,
@@ -51,16 +53,14 @@
         ['phpstan-template', 'phpstan-template-covariant', 'phpstan-extends', 'phpstan-implements', 'phpstan-var', 'psalm-var', 'phpstan-param', 'psalm-param', 'phpstan-return', 'psalm-return'],
         ['psalm-suppress', 'phpstan-ignore-next-line'],
         ['Assert\\*'],
-        ['ODM\\*'],
-        ['ORM\\*'],
     ]],
-    'php_unit_strict' => true,
-    'php_unit_test_case_static_method_calls' => true,
     'php_unit_data_provider_name' => true,
     'php_unit_data_provider_return_type' => true,
+    'php_unit_strict' => true,
+    'php_unit_test_case_static_method_calls' => true,
     'phpdoc_to_comment' => ['ignored_tags' => ['psalm-suppress', 'phpstan-var']],
-    'single_line_throw' => false,
     'static_lambda' => true,
+    'single_line_throw' => false,
     'strict_comparison' => true,
     'strict_param' => true,
     'void_return' => false,

diff --git a/src/Controller/CRUDController.php b/src/Controller/CRUDController.php
@@ -1457,7 +1457,8 @@ final protected function assertObjectExists(Request $request, bool $strict = fal
                         $admin->getClassnameLabel(),
                         $objectId
                     ));
-                } elseif (null === $object) {
+                }
+                if (null === $object) {
                     /** @phpstan-var T $object */
                     $object = $adminObject;
                 }

diff --git a/src/FieldDescription/BaseFieldDescription.php b/src/FieldDescription/BaseFieldDescription.php
@@ -416,7 +416,8 @@ final protected function getFieldValue(?object $object, string $fieldName)
         $accessor = $this->getOption('accessor', $fieldName);
         if (!\is_string($accessor) && \is_callable($accessor)) {
             return $accessor($object);
-        } elseif (!\is_string($accessor) && !$accessor instanceof PropertyPathInterface) {
+        }
+        if (!\is_string($accessor) && !$accessor instanceof PropertyPathInterface) {
             throw new \TypeError(sprintf(
                 'The option "accessor" must be a string, a callable or a %s, %s given.',
                 PropertyPathInterface::class,