Fix CS

nucleos · Feb 7, 2025 · 88b0442 · 88b0442
1 parent dda774c
commit 88b0442
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 20 deletions.
diff --git a/src/Test/AbstractFormHandlerTest.php b/src/Test/AbstractFormHandlerTest.php
@@ -165,7 +165,7 @@ private function equalToErrors(): Constraint
                 }
 
                 throw new AssertionFailedError(
-                    sprintf("Method 'addError' was not expected to be called with message '%s'", $error->getMessage())
+                    \sprintf("Method 'addError' was not expected to be called with message '%s'", $error->getMessage())
                 );
             }
 
@@ -193,7 +193,7 @@ private function checkUncalledErrors(): void
         foreach ($this->errors as $data) {
             if (0 === $data['count']) {
                 throw new AssertionFailedError(
-                    sprintf("Method 'addError' was expected to be called with message '%s' actually was not called", $data['message'])
+                    \sprintf("Method 'addError' was expected to be called with message '%s' actually was not called", $data['message'])
                 );
             }
         }

diff --git a/src/Validator/Constraints/BatchTimeAfter.php b/src/Validator/Constraints/BatchTimeAfter.php
@@ -52,12 +52,12 @@ final class BatchTimeAfter extends Constraint
      */
     public function __construct(
         array $options = [],
-        string $firstField = null,
-        string $secondField = null,
-        string $message = null,
-        string $emptyMessage = null,
-        bool $required = null,
-        array $groups = null,
+        ?string $firstField = null,
+        ?string $secondField = null,
+        ?string $message = null,
+        ?string $emptyMessage = null,
+        ?bool $required = null,
+        ?array $groups = null,
         mixed $payload = null,
     ) {
         if (null !== $firstField) {

diff --git a/src/Validator/Constraints/BatchTimeAfterValidator.php b/src/Validator/Constraints/BatchTimeAfterValidator.php
@@ -35,7 +35,7 @@ public function validate($value, Constraint $constraint): void
         }
 
         if (!\is_object($value)) {
-            throw new InvalidArgumentException(sprintf('Could not validate "%s"', \gettype($value)));
+            throw new InvalidArgumentException(\sprintf('Could not validate "%s"', \gettype($value)));
         }
 
         $firstDate  = $this->getFieldValue($value, $constraint->firstField);

diff --git a/src/Validator/Constraints/DateAfter.php b/src/Validator/Constraints/DateAfter.php
@@ -52,12 +52,12 @@ final class DateAfter extends Constraint
      */
     public function __construct(
         array $options = [],
-        string $firstField = null,
-        string $secondField = null,
-        string $message = null,
-        string $emptyMessage = null,
-        bool $required = null,
-        array $groups = null,
+        ?string $firstField = null,
+        ?string $secondField = null,
+        ?string $message = null,
+        ?string $emptyMessage = null,
+        ?bool $required = null,
+        ?array $groups = null,
         mixed $payload = null,
     ) {
         if (null !== $firstField) {

diff --git a/src/Validator/Constraints/DateAfterValidator.php b/src/Validator/Constraints/DateAfterValidator.php
@@ -34,7 +34,7 @@ public function validate($value, Constraint $constraint): void
         }
 
         if (!\is_object($value)) {
-            throw new InvalidArgumentException(sprintf('Could not validate "%s"', \gettype($value)));
+            throw new InvalidArgumentException(\sprintf('Could not validate "%s"', \gettype($value)));
         }
 
         $firstFieldName  = $constraint->firstField;

diff --git a/tests/Validator/Constraints/BatchTimeAfterTest.php b/tests/Validator/Constraints/BatchTimeAfterTest.php
@@ -21,7 +21,7 @@ final class BatchTimeAfterTest extends TestCase
     public function testItIsNotInstantiableWithMissingFirstField(): void
     {
         $this->expectException(MissingOptionsException::class);
-        $this->expectExceptionMessageMatches(sprintf('#^%s#', preg_quote('The options "firstField" must be set for constraint', '#')));
+        $this->expectExceptionMessageMatches(\sprintf('#^%s#', preg_quote('The options "firstField" must be set for constraint', '#')));
 
         new BatchTimeAfter([
             'secondField' => 'first',
@@ -31,7 +31,7 @@ public function testItIsNotInstantiableWithMissingFirstField(): void
     public function testItIsNotInstantiableWithMissingSecondField(): void
     {
         $this->expectException(MissingOptionsException::class);
-        $this->expectExceptionMessageMatches(sprintf('#^%s#', preg_quote('The options "secondField" must be set for constraint', '#')));
+        $this->expectExceptionMessageMatches(\sprintf('#^%s#', preg_quote('The options "secondField" must be set for constraint', '#')));
 
         new BatchTimeAfter([
             'firstField' => 'first',

diff --git a/tests/Validator/Constraints/DateAfterTest.php b/tests/Validator/Constraints/DateAfterTest.php
@@ -21,7 +21,7 @@ final class DateAfterTest extends TestCase
     public function testItIsNotInstantiableWithMissingFirstField(): void
     {
         $this->expectException(MissingOptionsException::class);
-        $this->expectExceptionMessageMatches(sprintf('#^%s#', preg_quote('The options "firstField" must be set for constraint', '#')));
+        $this->expectExceptionMessageMatches(\sprintf('#^%s#', preg_quote('The options "firstField" must be set for constraint', '#')));
 
         new DateAfter([
             'secondField' => 'first',
@@ -31,7 +31,7 @@ public function testItIsNotInstantiableWithMissingFirstField(): void
     public function testItIsNotInstantiableWithMissingSecondField(): void
     {
         $this->expectException(MissingOptionsException::class);
-        $this->expectExceptionMessageMatches(sprintf('#^%s#', preg_quote('The options "secondField" must be set for constraint', '#')));
+        $this->expectExceptionMessageMatches(\sprintf('#^%s#', preg_quote('The options "secondField" must be set for constraint', '#')));
 
         new DateAfter([
             'firstField' => 'first',