Validator: do not require UploadControl instance for file upload related validations

Author: xificurk

src/Forms/Validator.php

@@ -323,9 +323,10 @@ public static function validateFloat(IControl $control): bool
 	/**
 	 * Is file size in limit?
 	 */
-	public static function validateFileSize(Controls\UploadControl $control, $limit): bool
+	public static function validateFileSize(IControl $control, $limit): bool
 	{
 		foreach (static::toArray($control->getValue()) as $file) {
+			Validators::assert($file, Nette\Http\FileUpload::class, 'control value');
 			if ($file->getSize() > $limit || $file->getError() === UPLOAD_ERR_INI_SIZE) {
 				return false;
 			}
@@ -338,10 +339,11 @@ public static function validateFileSize(Controls\UploadControl $control, $limit)
 	 * Has file specified mime type?
 	 * @param  string|string[]  $mimeType
 	 */
-	public static function validateMimeType(Controls\UploadControl $control, $mimeType): bool
+	public static function validateMimeType(IControl $control, $mimeType): bool
 	{
 		$mimeTypes = is_array($mimeType) ? $mimeType : explode(',', $mimeType);
 		foreach (static::toArray($control->getValue()) as $file) {
+			Validators::assert($file, Nette\Http\FileUpload::class, 'control value');
 			$type = strtolower($file->getContentType());
 			if (!in_array($type, $mimeTypes, true) && !in_array(preg_replace('#/.*#', '/*', $type), $mimeTypes, true)) {
 				return false;
@@ -354,9 +356,10 @@ public static function validateMimeType(Controls\UploadControl $control, $mimeTy
 	/**
 	 * Is file image?
 	 */
-	public static function validateImage(Controls\UploadControl $control): bool
+	public static function validateImage(IControl $control): bool
 	{
 		foreach (static::toArray($control->getValue()) as $file) {
+			Validators::assert($file, Nette\Http\FileUpload::class, 'control value');
 			if (!$file->isImage()) {
 				return false;
 			}

tests/Forms/Validator.customControl.phpt

@@ -8,6 +8,8 @@ declare(strict_types=1);
 
 use Nette\Forms\Form;
 use Nette\Forms\Validator;
+use Nette\Http\FileUpload;
+use Nette\Utils\AssertionException;
 use Tester\Assert;
 
 
@@ -74,3 +76,47 @@ test(function () { // filled, blank
 	Assert::true(Validator::validateFilled($input));
 	Assert::false(Validator::validateBlank($input));
 });
+
+
+test(function () { // file upload related validators
+	$input = new CustomControl(new FileUpload([
+		'name' => 'foo',
+		'size' => 1,
+		'tmp_name' => __FILE__,
+		'error' => UPLOAD_ERR_OK
+	]));
+	Assert::true(Validator::validateFileSize($input, 42));
+	Assert::true(Validator::validateMimeType($input, ['text/x-php']));
+	Assert::false(Validator::validateImage($input));
+
+	$input = new CustomControl(new FileUpload([
+		'name' => 'foo',
+		'size' => 100,
+		'tmp_name' => __DIR__ . '/files/logo.gif',
+		'error' => UPLOAD_ERR_OK
+	]));
+	Assert::false(Validator::validateFileSize($input, 42));
+	Assert::false(Validator::validateMimeType($input, ['text/x-php']));
+	Assert::true(Validator::validateImage($input));
+
+	Assert::exception(
+		function () : void {
+			Assert::false(Validator::validateFileSize(new CustomControl('foo'), 42));
+		},
+		AssertionException::class
+	);
+
+	Assert::exception(
+		function () : void {
+			Assert::false(Validator::validateMimeType(new CustomControl('foo'), ['plain/text']));
+		},
+		AssertionException::class
+	);
+
+	Assert::exception(
+		function () : void {
+			Assert::false(Validator::validateImage(new CustomControl('foo')));
+		},
+		AssertionException::class
+	);
+});