ZipArchiveAdapter::close() - Code Metrics - Inspection of "Merge pull request #9 from PHPOffice/issuePHPPrese..." - PHPOffice/Common - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 72e4e7...fc5243 )

by Franck

created 2016-03-03 15:02 UTC

ZipArchiveAdapter::close() A

↳ Parent: ZipArchiveAdapter

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	7
rs	9.4285
cc	2
eloc	4
nc	2
nop	0

<?php

namespace PhpOffice\Common\Adapter\Zip;

use ZipArchive;

class ZipArchiveAdapter implements ZipInterface
{
    /**
     * @var ZipArchive
     */
    protected $oZipArchive;

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

    /**
     * @param string $filename
     * @throws \Exception Could not open $this->filename for writing.
     * @return mixed
     */
    public function open($filename)
    {
        $this->filename = $filename;
        $this->oZipArchive = new ZipArchive();

        if ($this->oZipArchive->open($this->filename, ZipArchive::OVERWRITE) === true) {
            return $this;
        }
        if ($this->oZipArchive->open($this->filename, ZipArchive::CREATE) === true) {
            return $this;
        }
        throw new \Exception("Could not open $this->filename for writing.");
    }

    /**
     * @return $this
     * @throws \Exception Could not close zip file $this->filename.
     */
    public function close()
    {
        if ($this->oZipArchive->close() === false) {
            throw new \Exception("Could not close zip file $this->filename.");
        }
        return $this;
    }

    /**
     * @param $localname
     * @param $contents
     * @return bool
     */
    public function addFromString($localname, $contents)
    {
        return $this->oZipArchive->addFromString($localname, $contents);
    }
}


1			<?php
2
3			namespace PhpOffice\Common\Adapter\Zip;
4
5			use ZipArchive;
6
7			class ZipArchiveAdapter implements ZipInterface
8			{
9			/**
10			* @var ZipArchive
11			*/
12			protected $oZipArchive;
13
14			/**
15			* @var string
16			*/
17			protected $filename;
18
19			/**
20			* @param string $filename
21			* @throws \Exception Could not open $this->filename for writing.
22			* @return mixed
23			*/
24			public function open($filename)
25			{
26			$this->filename = $filename;
27			$this->oZipArchive = new ZipArchive();
28
29			if ($this->oZipArchive->open($this->filename, ZipArchive::OVERWRITE) === true) {
30			return $this;
31			}
32			if ($this->oZipArchive->open($this->filename, ZipArchive::CREATE) === true) {
33			return $this;
34			}
35			throw new \Exception("Could not open $this->filename for writing.");
36			}
37
38			/**
39			* @return $this
40			* @throws \Exception Could not close zip file $this->filename.
41			*/
42			public function close()
43			{
44			if ($this->oZipArchive->close() === false) {
45			throw new \Exception("Could not close zip file $this->filename.");
46			}
47			return $this;
48			}
49
50			/**
51			* @param $localname
52			* @param $contents
53			* @return bool
54			*/
55			public function addFromString($localname, $contents)
56			{
57			return $this->oZipArchive->addFromString($localname, $contents);
58			}
59			}
60

PHPOffice / Common

Push — develop ( 72e4e7...fc5243 )

ZipArchiveAdapter::close() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like