[Validator] fixed coding standard issues

mdlutz24 · mdlutz24 · commit 59c08eda702b · 2019-04-11T19:32:11.000-07:00
diff --git a/src/Symfony/Component/Validator/ConstraintViolation.php b/src/Symfony/Component/Validator/ConstraintViolation.php
@@ -53,8 +53,8 @@ class ConstraintViolation implements ConstraintViolationInterface
      */
     public function __construct($message, ?string $messageTemplate, array $parameters, $root, ?string $propertyPath, $invalidValue, int $plural = null, $code = null, Constraint $constraint = null, $cause = null)
     {
-        if (!(NULL === $message || is_string($message) || (is_object($message) && method_exists($message, '__toString' )))) {
-            throw new InvalidArgumentException("Constraint violation message should be a string or an object which implements the __toString() method.");
+        if (!(null === $message || \is_string($message) || (\is_object($message) && method_exists($message, '__toString')))) {
+            throw new InvalidArgumentException('Constraint violation message should be a string or an object which implements the __toString() method.');
         }
         $this->message = $message;
         $this->messageTemplate = $messageTemplate;
diff --git a/src/Symfony/Component/Validator/Tests/ConstraintViolationTest.php b/src/Symfony/Component/Validator/Tests/ConstraintViolationTest.php
@@ -35,7 +35,7 @@ public function testToStringHandlesArrays()
     Array
 EOF;
 
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
     }
 
     public function testToStringHandlesArrayRoots()
@@ -54,7 +54,7 @@ public function testToStringHandlesArrayRoots()
     42 cannot be used here
 EOF;
 
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
     }
 
     public function testToStringHandlesCodes()
@@ -75,7 +75,7 @@ public function testToStringHandlesCodes()
     42 cannot be used here (code 0)
 EOF;
 
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
     }
 
     public function testToStringOmitsEmptyCodes()
@@ -96,7 +96,7 @@ public function testToStringOmitsEmptyCodes()
             null
         );
 
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
 
         $violation = new ConstraintViolation(
             '42 cannot be used here',
@@ -109,7 +109,7 @@ public function testToStringOmitsEmptyCodes()
             ''
         );
 
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
     }
 
     public function testMessageCanBeStringableObject()
@@ -128,7 +128,7 @@ public function testMessageCanBeStringableObject()
 Root.property.path:
     toString
 EOF;
-        $this->assertSame($expected, (string)$violation);
+        $this->assertSame($expected, (string) $violation);
         $this->assertSame($message, $violation->getMessage());
     }
 
@@ -157,5 +157,4 @@ public function testMessageObjectMustBeStringable()
             null
         );
     }
-
 }

Original file line number	Diff line number	Diff line change
`@@ -53,8 +53,8 @@ class ConstraintViolation implements ConstraintViolationInterface`
`53`	`53`	`*/`
`54`	`54`	`public function __construct($message, ?string $messageTemplate, array $parameters, $root, ?string $propertyPath, $invalidValue, int $plural = null, $code = null, Constraint $constraint = null, $cause = null)`
`55`	`55`	`{`
`56`		`- if (!(NULL === $message \|\| is_string($message) \|\| (is_object($message) && method_exists($message, '__toString' )))) {`
`57`		`- throw new InvalidArgumentException("Constraint violation message should be a string or an object which implements the __toString() method.");`
	`56`	`+ if (!(null === $message \|\| \is_string($message) \|\| (\is_object($message) && method_exists($message, '__toString')))) {`
	`57`	`+ throw new InvalidArgumentException('Constraint violation message should be a string or an object which implements the __toString() method.');`
`58`	`58`	`}`
`59`	`59`	`$this->message = $message;`
`60`	`60`	`$this->messageTemplate = $messageTemplate;`