ConfigWriter::setCurrentConfig() - Code Metrics - Inspection of "Throw exception with more useful message when tryi..." - spekkionu/laravel-assetcachebuster - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4fe946...35cb72 )

by Jonathan

created 2017-10-17 16:45 UTC

ConfigWriter::setCurrentConfig() A

↳ Parent: ConfigWriter

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

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

<?php
namespace Spekkionu\Assetcachebuster\Writer;

use Illuminate\Filesystem\Filesystem;

class ConfigWriter implements WriterInterface
{
    /**
     * @var Filesystem
     */
    private $filesystem;
    /**
     * @var
     */
    private $config_path;

    /**
     * ConfigWriter constructor.
     * @param Filesystem $filesystem
     * @param string $config_path
     */
    public function __construct(Filesystem $filesystem, $config_path)
    {
        $this->filesystem = $filesystem;
        $this->config_path = $config_path;
    }

    /**
     * Get the key file and contents.
     *
     * @return array
     */
    protected function getConfigFile()
    {
        return $this->config_path . DIRECTORY_SEPARATOR . "assetcachebuster.php";
    }

    /**
     * Check if the config file exists
     * 
     * @return bool
     */
    protected function configExists()
    {
        return $this->filesystem->exists($this->getConfigFile());
    }

    /**
     * Returns current config as string
     *
     * @return string
     */
    public function getCurrentConfig()
    {
        if (!$this->configExists()) {
            throw new \InvalidArgumentException('The config file does not exist. Did you run the vendor:publish command?');
        }
        return $this->filesystem->get($this->getConfigFile());
    }

    /**
     * Saves config as current config
     * @param string $content
     * @return void
     */
    public function setCurrentConfig($content)
    {
        if (!$this->configExists()) {
            throw new \InvalidArgumentException('The config file does not exist. Did you run the vendor:publish command?');
        }
        $this->filesystem->put($this->getConfigFile(), $content);
    }
}


1		<?php
2		namespace Spekkionu\Assetcachebuster\Writer;
3
4		use Illuminate\Filesystem\Filesystem;
5
6		class ConfigWriter implements WriterInterface
7		{
8		/**
9		* @var Filesystem
10		*/
11		private $filesystem;
12		/**
13		* @var
14		*/
15		private $config_path;
16
17		/**
18		* ConfigWriter constructor.
19		* @param Filesystem $filesystem
20		* @param string $config_path
21		*/
22	4	public function __construct(Filesystem $filesystem, $config_path)
23		{
24	4	$this->filesystem = $filesystem;
25	4	$this->config_path = $config_path;
26	4	}
27
28		/**
29		* Get the key file and contents.
30		*
31		* @return array
32		*/
33	4	protected function getConfigFile()
34		{
35	4	return $this->config_path . DIRECTORY_SEPARATOR . "assetcachebuster.php";
36		}
37
38		/**
39		* Check if the config file exists
40		*
41		* @return bool
42		*/
43	4	protected function configExists()
44		{
45	4	return $this->filesystem->exists($this->getConfigFile());
46		}
47
48		/**
49		* Returns current config as string
50		*
51		* @return string
52		*/
53	2	public function getCurrentConfig()
54		{
55	2	if (!$this->configExists()) {
56	1	throw new \InvalidArgumentException('The config file does not exist. Did you run the vendor:publish command?');
57		}
58	1	return $this->filesystem->get($this->getConfigFile());
59		}
60
61		/**
62		* Saves config as current config
63		* @param string $content
64		* @return void
65		*/
66	2	public function setCurrentConfig($content)
67		{
68	2	if (!$this->configExists()) {
69	1	throw new \InvalidArgumentException('The config file does not exist. Did you run the vendor:publish command?');
70		}
71	1	$this->filesystem->put($this->getConfigFile(), $content);
72	1	}
73		}
74

spekkionu / laravel-assetcachebuster

Push — master ( 4fe946...35cb72 )

ConfigWriter::setCurrentConfig() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like