Allow strict integer validation (laravel#56503)

zll600 · Leigang Zhang · web-flow · commit a3267a351d0c · 2025-08-03T09:52:14.000-05:00
Co-authored-by: Leigang Zhang &lt;leigang.zhang@rightcapital.com&gt;
diff --git a/src/Illuminate/Validation/Concerns/ValidatesAttributes.php b/src/Illuminate/Validation/Concerns/ValidatesAttributes.php
@@ -1554,10 +1554,15 @@ public function validateInArrayKeys($attribute, $value, $parameters)
      *
      * @param  string  $attribute
      * @param  mixed  $value
+     * @param  array{0: 'strict'}  $parameters
      * @return bool
      */
-    public function validateInteger($attribute, $value)
+    public function validateInteger($attribute, $value, array $parameters)
     {
+        if (($parameters[0] ?? null) === 'strict') {
+            return is_int($value);
+        }
+
         return filter_var($value, FILTER_VALIDATE_INT) !== false;
     }
 
diff --git a/tests/Validation/ValidationValidatorTest.php b/tests/Validation/ValidationValidatorTest.php
@@ -3283,6 +3283,28 @@ public function testValidateInteger()
         $this->assertTrue($v->passes());
     }
 
+    public function testValidateIntegerStrict()
+    {
+        $trans = $this->getIlluminateArrayTranslator();
+        $v = new Validator($trans, ['foo' => 'asdad'], ['foo' => 'Integer:strict']);
+        $this->assertFalse($v->passes());
+
+        $v = new Validator($trans, ['foo' => '1.23'], ['foo' => 'Integer:strict']);
+        $this->assertFalse($v->passes());
+
+        $v = new Validator($trans, ['foo' => '-1'], ['foo' => 'Integer:strict']);
+        $this->assertFalse($v->passes());
+
+        $v = new Validator($trans, ['foo' => '1'], ['foo' => 'Integer:strict']);
+        $this->assertFalse($v->passes());
+
+        $v = new Validator($trans, ['foo' => 0.1], ['foo' => 'Integer:strict']);
+        $this->assertFalse($v->passes());
+
+        $v = new Validator($trans, ['foo' => 1], ['foo' => 'Integer:strict']);
+        $this->assertTrue($v->passes());
+    }
+
     public function testValidateDecimal()
     {
         $trans = $this->getIlluminateArrayTranslator();

Original file line number	Diff line number	Diff line change
`@@ -1554,10 +1554,15 @@ public function validateInArrayKeys($attribute, $value, $parameters)`
`1554`	`1554`	`*`
`1555`	`1555`	`* @param string $attribute`
`1556`	`1556`	`* @param mixed $value`
	`1557`	`+ * @param array{0: 'strict'} $parameters`
`1557`	`1558`	`* @return bool`
`1558`	`1559`	`*/`
`1559`		`- public function validateInteger($attribute, $value)`
	`1560`	`+ public function validateInteger($attribute, $value, array $parameters)`
`1560`	`1561`	`{`
	`1562`	`+ if (($parameters[0] ?? null) === 'strict') {`
	`1563`	`+ return is_int($value);`
	`1564`	`+ }`
	`1565`	`+`
`1561`	`1566`	`return filter_var($value, FILTER_VALIDATE_INT) !== false;`
`1562`	`1567`	`}`
`1563`	`1568`