UploadControl: automatically sets MAX_FILE_SIZE [Closes #238]

Author: dg

src/Forms/Controls/UploadControl.php

@@ -34,6 +34,7 @@ public function __construct($label = null, bool $multiple = false)
 		$this->control->multiple = $multiple;
 		$this->setOption('type', 'file');
 		$this->addRule([$this, 'isOk'], Forms\Validator::$messages[self::VALID]);
+		$this->addRule(Form::MAX_FILE_SIZE, null, Forms\Helpers::iniGetSize('upload_max_filesize'));
 
 		$this->monitor(Form::class, function (Form $form): void {
 			if (!$form->isMethod('post')) {
@@ -108,6 +109,8 @@ public function addRule($validator, $errorMessage = null, $arg = null)
 			$this->control->accept = implode(', ', FileUpload::IMAGE_MIME_TYPES);
 		} elseif ($validator === Form::MIME_TYPE) {
 			$this->control->accept = implode(', ', (array) $arg);
+		} elseif ($validator === Form::MAX_FILE_SIZE) {
+			$this->getRules()->removeRule($validator);
 		}
 		return parent::addRule($validator, $errorMessage, $arg);
 	}

tests/Forms/Controls.UploadControl.render.phpt

@@ -32,15 +32,15 @@ test(function () {
 	Assert::same('<label for="frm-file">Another label</label>', (string) $input->getLabel('Another label'));
 
 	Assert::type(Html::class, $input->getControl());
-	Assert::same('<input type="file" name="file" id="frm-file">', (string) $input->getControl());
+	Assert::match('<input type="file" name="file" id="frm-file" data-nette-rules=\'[{"op":":fileSize","msg":"The size of the uploaded file can be up to %d% bytes.","arg":%d%}]\'>', (string) $input->getControl());
 });
 
 
 test(function () { // multiple
 	$form = new Form;
 	$input = $form->addMultiUpload('file', 'Label');
 
-	Assert::same('<input type="file" name="file[]" multiple id="frm-file">', (string) $input->getControl());
+	Assert::match('<input type="file" name="file[]" multiple id="frm-file" data-nette-rules=%a%>', (string) $input->getControl());
 });
 
 
@@ -58,21 +58,20 @@ test(function () { // Html with translator
 test(function () { // validation rules
 	$form = new Form;
 	$input = $form->addUpload('file')->setRequired('required');
-
-	Assert::same('<input type="file" name="file" id="frm-file" required data-nette-rules=\'[{"op":":filled","msg":"required"}]\'>', (string) $input->getControl());
+	Assert::match('<input type="file" name="file" id="frm-file" required data-nette-rules=\'[{"op":":filled","msg":"required"},{"op":":fileSize",%a%}]\'>', (string) $input->getControl());
 });
 
 
 test(function () { // accepted files
 	$form = new Form;
 	$input = $form->addUpload('file1')->addRule(Form::MIME_TYPE, null, 'image/*');
-	Assert::same('<input type="file" name="file1" accept="image/*" id="frm-file1" data-nette-rules=\'[{"op":":mimeType","msg":"The uploaded file is not in the expected format.","arg":"image/*"}]\'>', (string) $input->getControl());
+	Assert::match('<input type="file" name="file1" accept="image/*" id="frm-file1" data-nette-rules=\'[{"op":":fileSize",%a%},{"op":":mimeType","msg":"The uploaded file is not in the expected format.","arg":"image/*"}]\'>', (string) $input->getControl());
 
 	$input = $form->addUpload('file2')->addRule(Form::MIME_TYPE, null, ['image/*', 'text/html']);
-	Assert::same('<input type="file" name="file2" accept="image/*, text/html" id="frm-file2" data-nette-rules=\'[{"op":":mimeType","msg":"The uploaded file is not in the expected format.","arg":["image/*","text/html"]}]\'>', (string) $input->getControl());
+	Assert::match('<input type="file" name="file2" accept="image/*, text/html" id="frm-file2" data-nette-rules=\'[{"op":":fileSize",%a%},{"op":":mimeType","msg":"The uploaded file is not in the expected format.","arg":["image/*","text/html"]}]\'>', (string) $input->getControl());
 
 	$input = $form->addUpload('file3')->addRule(Form::IMAGE);
-	Assert::same('<input type="file" name="file3" accept="image/gif, image/png, image/jpeg, image/webp" id="frm-file3" data-nette-rules=\'[{"op":":image","msg":"The uploaded file must be image in format JPEG, GIF, PNG or WebP."}]\'>', (string) $input->getControl());
+	Assert::match('<input type="file" name="file3" accept="image/gif, image/png, image/jpeg, image/webp" id="frm-file3" data-nette-rules=\'[{"op":":fileSize",%a%},{"op":":image","msg":"The uploaded file must be image in format JPEG, GIF, PNG or WebP."}]\'>', (string) $input->getControl());
 });
 
 
@@ -81,7 +80,7 @@ test(function () { // container
 	$container = $form->addContainer('container');
 	$input = $container->addUpload('file');
 
-	Assert::same('<input type="file" name="container[file]" id="frm-container-file">', (string) $input->getControl());
+	Assert::match('<input type="file" name="container[file]" id="frm-container-file" data-nette-rules=%a%>', (string) $input->getControl());
 });
 
 

tests/Forms/Forms.renderer.1.expect

@@ -110,7 +110,7 @@
 <tr>
 	<th><label for="frm-avatar">Picture:</label></th>
 
-	<td><input type="file" name="avatar" id="frm-avatar" data-nette-rules='[{"op":":filled","rules":[{"op":":image","msg":"Uploaded file is not image"}],"control":"avatar"}]' class="text"></td>
+	<td><input type="file" name="avatar" id="frm-avatar" data-nette-rules='[{"op":":fileSize","msg":"The size of the uploaded file can be up to %d% bytes.","arg":%d%},{"op":":filled","rules":[{"op":":image","msg":"Uploaded file is not image"}],"control":"avatar"}]' class="text"></td>
 </tr>
 
 <tr>

tests/Forms/Forms.renderer.2.expect

@@ -98,7 +98,7 @@
 
 	<dt><label for="frm-avatar">Picture:</label></dt>
 
-	<dd><input type="file" name="avatar" id="frm-avatar" class="text"></dd>
+	<dd><input type="file" name="avatar" id="frm-avatar" data-nette-rules='[{"op":":fileSize","msg":"The size of the uploaded file can be up to %d% bytes.","arg":%d%}]' class="text"></dd>