Nip_File_Upload::upload() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/bytic/..." - bytic/filesystem - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 1b90b9...7956e0 )

by Gabriel

created 2020-04-23 14:18 UTC

Nip_File_Upload::upload() A

↳ Parent: Nip_File_Upload

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	1
eloc	1
c	1
b	0
f	1
nc	1
nop	1
dl	0
loc	3
rs	10

<?php

class Nip_File_Upload extends Nip_File
{
    protected $_error;
    protected $_tmp_name;

    public function __construct($data = [])
    {
        if ($data) {
            $this->_name = $data['name'];
            $this->_tmp_name = $data['tmp_name'];

            parent::__construct(TMP_PATH.$this->_tmp_name);

        }
    }

    public function valid()
    {
        $result = false;


        $max_upload = ini_get('post_max_size');
        $unit = strtoupper(substr($max_upload, -1));
        $multiplier = ($unit == 'M') ? 1048576 : (($unit == 'K') ? 1024 : (($unit == 'G') ? 1073741824 : 1));

        if ($max_upload && ((int) $_SERVER['CONTENT_LENGTH'] > $multiplier * (int) $max_upload)) {
            return self::ERROR_MAX_POST_SIZE;
        }

        if (!$this->getPath()) {
            return false;
        } elseif (isset($this->error) && $this->error != 0) {
            return false;
        } elseif (!isset($this->_tmp_name) || !@is_uploaded_file($this->_tmp_name)) {
            return false;
        } elseif (!isset($this->_name)) {
            return false;
        }

        return true;
    }

    public function upload($path)
    {
        return $this->move($path);
    }
}


1			<?php
2
3			class Nip_File_Upload extends Nip_File
4			{
5			protected $_error;
6			protected $_tmp_name;
7
8			public function __construct($data = [])
9			{
10			if ($data) {
11			$this->_name = $data['name'];
12			$this->_tmp_name = $data['tmp_name'];
13
14			parent::__construct(TMP_PATH.$this->_tmp_name);
			0 ignored issues – show Bug introduced 2020-04-23 14:22 UTC by Report Bug Copy Issue Report The constant `TMP_PATH` was not found. Maybe you did not declare it correctly or list all dependencies? Loading history...
15			}
16			}
17
18			public function valid()
19			{
20			$result = false;
			0 ignored issues – show Unused Code introduced 2020-04-23 14:22 UTC by Report Bug Copy Issue Report The assignment to `$result` is dead and can be removed. Loading history...
21
22			$max_upload = ini_get('post_max_size');
23			$unit = strtoupper(substr($max_upload, -1));
24			$multiplier = ($unit == 'M') ? 1048576 : (($unit == 'K') ? 1024 : (($unit == 'G') ? 1073741824 : 1));
25
26			if ($max_upload && ((int) $_SERVER['CONTENT_LENGTH'] > $multiplier * (int) $max_upload)) {
27			return self::ERROR_MAX_POST_SIZE;
28			}
29
30			if (!$this->getPath()) {
31			return false;
32			} elseif (isset($this->error) && $this->error != 0) {
33			return false;
34			} elseif (!isset($this->_tmp_name) \|\| !@is_uploaded_file($this->_tmp_name)) {
35			return false;
36			} elseif (!isset($this->_name)) {
37			return false;
38			}
39
40			return true;
41			}
42
43			public function upload($path)
44			{
45			return $this->move($path);
46			}
47			}
48

bytic / filesystem

Push — master ( 1b90b9...7956e0 )

Nip_File_Upload::upload() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like