PclZipAdapter::addFromString() - Code Metrics - Inspection of "Merge pull request #24 from troosan/update_phpunit" - PHPOffice/Common - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 522e3f...10c5c8 )

by Franck

created 2018-07-13 13:56 UTC

PclZipAdapter::addFromString() A

↳ Parent: PclZipAdapter

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	15
ccs	0
cts	10
cp	0
rs	9.7666
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	6

<?php
namespace PhpOffice\Common\Adapter\Zip;

use PclZip;

class PclZipAdapter implements ZipInterface
{
    /**
     * @var PclZip
     */
    protected $oPclZip;

    /**
     * @var string
     */
    protected $tmpDir;

    public function open($filename)
    {
        $this->oPclZip = new PclZip($filename);
        $this->tmpDir = sys_get_temp_dir();
        return $this;
    }

    public function close()
    {
        return $this;
    }

    public function addFromString($localname, $contents)
    {
        $pathData = pathinfo($localname);

        $hFile = fopen($this->tmpDir.'/'.$pathData['basename'], "wb");
        fwrite($hFile, $contents);
        fclose($hFile);

        $res = $this->oPclZip->add($this->tmpDir.'/'.$pathData['basename'], PCLZIP_OPT_REMOVE_PATH, $this->tmpDir, PCLZIP_OPT_ADD_PATH, $pathData['dirname']);
        if ($res == 0) {
            throw new \Exception("Error zipping files : " . $this->oPclZip->errorInfo(true));
        }
        unlink($this->tmpDir.'/'.$pathData['basename']);
        return $this;
    }
}


1		<?php
2		namespace PhpOffice\Common\Adapter\Zip;
3
4		use PclZip;
5
6		class PclZipAdapter implements ZipInterface
7		{
8		/**
9		* @var PclZip
10		*/
11		protected $oPclZip;
12
13		/**
14		* @var string
15		*/
16		protected $tmpDir;
17
18	2	public function open($filename)
19		{
20	2	$this->oPclZip = new PclZip($filename);
21	2	$this->tmpDir = sys_get_temp_dir();
22	2	return $this;
23		}
24
25	1	public function close()
26		{
27	1	return $this;
28		}
29
30		public function addFromString($localname, $contents)
31		{
32		$pathData = pathinfo($localname);
33
34		$hFile = fopen($this->tmpDir.'/'.$pathData['basename'], "wb");
35		fwrite($hFile, $contents);
36		fclose($hFile);
37
38		$res = $this->oPclZip->add($this->tmpDir.'/'.$pathData['basename'], PCLZIP_OPT_REMOVE_PATH, $this->tmpDir, PCLZIP_OPT_ADD_PATH, $pathData['dirname']);
39		if ($res == 0) {
40		throw new \Exception("Error zipping files : " . $this->oPclZip->errorInfo(true));
41		}
42		unlink($this->tmpDir.'/'.$pathData['basename']);
43		return $this;
44		}
45		}
46

PHPOffice / Common

Push — develop ( 522e3f...10c5c8 )

PclZipAdapter::addFromString() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like