ConfigHashReplacer::writeHash() - Code Metrics - Inspection of "add tests" - spekkionu/laravel-assetcachebuster - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 635436...706b38 )

by Jonathan

created 2017-09-01 03:19 UTC

ConfigHashReplacer::writeHash() A

↳ Parent: ConfigHashReplacer

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	17
ccs	12
cts	12
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	12
nc	2
nop	2
crap	2

<?php
namespace Spekkionu\Assetcachebuster\HashReplacer;

use Spekkionu\Assetcachebuster\Writer\WriterInterface;
use Spekkionu\Assetcachebuster\Assetcachebuster as CacheBuster;

class ConfigHashReplacer implements HashReplacerInterface
{
    /**
     * @var WriterInterface
     */
    private $writer;

    /**
     * @var CacheBuster
     */
    private $cacheBuster;

    /**
     * HashReplacer constructor.
     * @param CacheBuster $cacheBuster
     * @param WriterInterface $writer
     */
    public function __construct(CacheBuster $cacheBuster, WriterInterface $writer)
    {
        $this->writer = $writer;
        $this->cacheBuster = $cacheBuster;
    }

    /**
     * Generate and save new hash
     * @return string New Hash
     * @throws \Exception
     */
    public function replaceHash()
    {
        $currentHash = $this->cacheBuster->getHash();
        $hash = $this->cacheBuster->generateHash();
        $this->writeHash($currentHash, $hash);
        $this->cacheBuster->setHash($hash);
        return $hash;
    }

    /**
     * @return string New hash
     * @throws \Exception
     */
    public function writeHash($currentHash, $hash)
    {
        $content = $this->writer->getCurrentConfig();
        $content = preg_replace(
            "/([\'\"]hash[\'\"].+?[\'\"])(" . preg_quote($currentHash, '/') . ")([\'\"].*)/",
            "'hash' => '" . $hash . "',",
            $content,
            1,
            $count
        );
        if ($count != 1) {
            throw new \RuntimeException("Could not find current hash key in config.");
        }

        $this->writer->setCurrentConfig($content);
        return $hash;
    }
}


1		<?php
2		namespace Spekkionu\Assetcachebuster\HashReplacer;
3
4		use Spekkionu\Assetcachebuster\Writer\WriterInterface;
5		use Spekkionu\Assetcachebuster\Assetcachebuster as CacheBuster;
6
7		class ConfigHashReplacer implements HashReplacerInterface
8		{
9		/**
10		* @var WriterInterface
11		*/
12		private $writer;
13
14		/**
15		* @var CacheBuster
16		*/
17		private $cacheBuster;
18
19		/**
20		* HashReplacer constructor.
21		* @param CacheBuster $cacheBuster
22		* @param WriterInterface $writer
23		*/
24	2	public function __construct(CacheBuster $cacheBuster, WriterInterface $writer)
25		{
26	2	$this->writer = $writer;
27	2	$this->cacheBuster = $cacheBuster;
28	2	}
29
30		/**
31		* Generate and save new hash
32		* @return string New Hash
33		* @throws \Exception
34		*/
35	2	public function replaceHash()
36		{
37	2	$currentHash = $this->cacheBuster->getHash();
38	2	$hash = $this->cacheBuster->generateHash();
39	2	$this->writeHash($currentHash, $hash);
40	1	$this->cacheBuster->setHash($hash);
41	1	return $hash;
42		}
43
44		/**
45		* @return string New hash
46		* @throws \Exception
47		*/
48	2	public function writeHash($currentHash, $hash)
49		{
50	2	$content = $this->writer->getCurrentConfig();
51	2	$content = preg_replace(
52	2	"/([\'\"]hash[\'\"].+?[\'\"])(" . preg_quote($currentHash, '/') . ")([\'\"].*)/",
53	2	"'hash' => '" . $hash . "',",
54	2	$content,
55	2	1,
56	2	$count
57		);
58	2	if ($count != 1) {
59	1	throw new \RuntimeException("Could not find current hash key in config.");
60		}
61
62	1	$this->writer->setCurrentConfig($content);
63	1	return $hash;
64		}
65		}
66

spekkionu / laravel-assetcachebuster

Push — master ( 635436...706b38 )

ConfigHashReplacer::writeHash() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like