minor #36033 [Validator][Range] Fix typos (przemyslaw-bogusz)

nicolas-grekas · nicolas-grekas · commit 9d07aa033dc0 · 2020-03-12T16:18:57.000+01:00
This PR was merged into the 4.4 branch. Discussion ---------- [Validator][Range] Fix typos | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no | Deprecations? | no | License | MIT Commits ------- a08d519 [Validator][Range] Fix typos
diff --git a/src/Symfony/Component/Validator/Constraints/Range.php b/src/Symfony/Component/Validator/Constraints/Range.php
@@ -65,7 +65,7 @@ public function __construct($options = null)
         parent::__construct($options);
 
         if (null === $this->min && null === $this->minPropertyPath && null === $this->max && null === $this->maxPropertyPath) {
-            throw new MissingOptionsException(sprintf('Either option "min", "minPropertyPath", "max" or "maxPropertyPath" must be given for constraint %s', __CLASS__), ['min', 'max']);
+            throw new MissingOptionsException(sprintf('Either option "min", "minPropertyPath", "max" or "maxPropertyPath" must be given for constraint %s', __CLASS__), ['min', 'minPropertyPath', 'max', 'maxPropertyPath']);
         }
     }
 }
diff --git a/src/Symfony/Component/Validator/Tests/Constraints/RangeTest.php b/src/Symfony/Component/Validator/Tests/Constraints/RangeTest.php
@@ -22,7 +22,7 @@ public function testThrowsConstraintExceptionIfBothMaxLimitAndPropertyPath()
         $this->expectException('Symfony\Component\Validator\Exception\ConstraintDefinitionException');
         $this->expectExceptionMessage('requires only one of the "max" or "maxPropertyPath" options to be set, not both.');
         new Range([
-            'max' => 'min',
+            'max' => 'max',
             'maxPropertyPath' => 'maxPropertyPath',
         ]);
     }

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public function __construct($options = null)`
`65`	`65`	`parent::__construct($options);`
`66`	`66`
`67`	`67`	`if (null === $this->min && null === $this->minPropertyPath && null === $this->max && null === $this->maxPropertyPath) {`
`68`		`- throw new MissingOptionsException(sprintf('Either option "min", "minPropertyPath", "max" or "maxPropertyPath" must be given for constraint %s', __CLASS__), ['min', 'max']);`
	`68`	`+ throw new MissingOptionsException(sprintf('Either option "min", "minPropertyPath", "max" or "maxPropertyPath" must be given for constraint %s', __CLASS__), ['min', 'minPropertyPath', 'max', 'maxPropertyPath']);`
`69`	`69`	`}`
`70`	`70`	`}`
`71`	`71`	`}`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public function testThrowsConstraintExceptionIfBothMaxLimitAndPropertyPath()`
`22`	`22`	`$this->expectException('Symfony\Component\Validator\Exception\ConstraintDefinitionException');`
`23`	`23`	`$this->expectExceptionMessage('requires only one of the "max" or "maxPropertyPath" options to be set, not both.');`
`24`	`24`	`new Range([`
`25`		`- 'max' => 'min',`
	`25`	`+ 'max' => 'max',`
`26`	`26`	`'maxPropertyPath' => 'maxPropertyPath',`
`27`	`27`	`]);`
`28`	`28`	`}`