ZipCompressor::compress() - Code Metrics - Inspection of "not use suffix in tempnam" - gpslab/compressor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 62d0c8...1d7e23 )

by Peter

created 2017-01-23 15:43 UTC

ZipCompressor::compress() A

↳ Parent: ZipCompressor

Complexity

Conditions	4
Paths	6

Size

Total Lines	16
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	4.8437

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	5
cts	8
cp	0.625
rs	9.2
c	0
b	0
f	0
cc	4
eloc	8
nc	6
nop	2
crap	4.8437

<?php
/**
 * GpsLab component.
 *
 * @author    Peter Gribanov <[email protected]>
 * @copyright Copyright (c) 2011, Peter Gribanov
 * @license   http://opensource.org/licenses/MIT
 */

namespace GpsLab\Component\Compressor;

class ZipCompressor implements CompressorInterface
{
    /**
     * @var \ZipArchive
     */
    protected $zip;

    /**
     * @param \ZipArchive $zip
     */
    public function __construct(\ZipArchive $zip)
    {
        $this->zip = $zip;
    }

    /**
     * @param string $source
     * @param string $target
     *
     * @return bool
     */
    public function compress($source, $target = '')
    {
        $target = $target ?: $source.'.zip';

        if ($this->zip->open($target) === false) {
            return false;
        }

        if ($this->zip->addFile($source, basename($source)) === false) {
            $this->zip->close();

            return false;
        }

        return $this->zip->close();
    }

    /**
     * @param string $source
     * @param string $target
     *
     * @return bool
     */
    public function uncompress($source, $target)
    {
        if ($this->zip->open($source) === false) {
            return false;
        }

        if ($this->zip->extractTo(dirname($target), basename($target)) === false) {
            $this->zip->close();

            return false;
        }

        return $this->zip->close();
    }
}


1		<?php
2		/**
3		* GpsLab component.
4		*
5		* @author Peter Gribanov <[email protected]>
6		* @copyright Copyright (c) 2011, Peter Gribanov
7		* @license http://opensource.org/licenses/MIT
8		*/
9
10		namespace GpsLab\Component\Compressor;
11
12		class ZipCompressor implements CompressorInterface
13		{
14		/**
15		* @var \ZipArchive
16		*/
17		protected $zip;
18
19		/**
20		* @param \ZipArchive $zip
21		*/
22	1	public function __construct(\ZipArchive $zip)
23		{
24	1	$this->zip = $zip;
25	1	}
26
27		/**
28		* @param string $source
29		* @param string $target
30		*
31		* @return bool
32		*/
33	1	public function compress($source, $target = '')
34		{
35	1	$target = $target ?: $source.'.zip';
36
37	1	if ($this->zip->open($target) === false) {
38		return false;
39		}
40
41	1	if ($this->zip->addFile($source, basename($source)) === false) {
42		$this->zip->close();
43
44		return false;
45		}
46
47	1	return $this->zip->close();
48		}
49
50		/**
51		* @param string $source
52		* @param string $target
53		*
54		* @return bool
55		*/
56	1	public function uncompress($source, $target)
57		{
58	1	if ($this->zip->open($source) === false) {
59		return false;
60		}
61
62	1	if ($this->zip->extractTo(dirname($target), basename($target)) === false) {
63		$this->zip->close();
64
65		return false;
66		}
67
68	1	return $this->zip->close();
69		}
70		}
71

gpslab / compressor

Push — master ( 62d0c8...1d7e23 )

ZipCompressor::compress() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like