FileUploadValidator   A
last analyzed

Complexity

Total Complexity 4

Size/Duplication

Total Lines 47
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 2

Importance

Changes 0
Metric Value
wmc 4
lcom 1
cbo 2
dl 0
loc 47
rs 10
c 0
b 0
f 0

2 Methods

Rating   Name   Duplication   Size   Complexity  
A validate() 0 10 2
A getErrorMessage() 0 4 2
1
<?php
2
3
namespace yiicod\fileupload\validators;
4
5
use yii\base\BaseObject;
6
use yiicod\fileupload\components\common\UploadedFile;
7
8
/**
9
 * Class FileUploadValidator
10
 *
11
 * @author Virchenko Maksim <[email protected]>
12
 *
13
 * @package yiicod\fileupload\validators
14
 */
15
class FileUploadValidator extends BaseObject implements ValidatorInterface
16
{
17
    /**
18
     * Error message
19
     *
20
     * @var string
21
     */
22
    public $messages = [
23
        1 => 'The uploaded file exceeds the upload_max_filesize directive in php.ini',
24
        2 => 'The uploaded file exceeds the MAX_FILE_SIZE directive that was specified in the HTML form',
25
        3 => 'The uploaded file was only partially uploaded',
26
        4 => 'No file was uploaded',
27
        6 => 'Missing a temporary folder',
28
        7 => 'Failed to write file to disk',
29
        8 => 'A PHP extension stopped the file upload',
30
    ];
31
32
    /**
33
     * Validate file
34
     *
35
     * @param UploadedFile $file
36
     *
37
     * @return bool
38
     */
39
    public function validate(UploadedFile &$file): bool
40
    {
41
        if ($file->error) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $file->error of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

In PHP, under loose comparison (like ==, or !=, or switch conditions), values of different types might be equal.

For string values, the empty string '' is a special case, in particular the following results might be unexpected:

''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
Loading history...
42
            $file->error = $this->getErrorMessage($file->error);
43
44
            return false;
45
        }
46
47
        return true;
48
    }
49
50
    /**
51
     * Get error message
52
     *
53
     * @param $error
54
     *
55
     * @return mixed
56
     */
57
    protected function getErrorMessage($error)
58
    {
59
        return isset($this->messages[$error]) ? $this->messages[$error] : $error;
60
    }
61
}
62