CollisionHandler::mergeJson() - Code Metrics - Inspection of "Merge pull request #3 from adhocore/develop" - adhocore/phint - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dfd004...b46837 )

by Jitendra

created 2017-10-29 10:58 UTC

CollisionHandler::mergeJson() A

↳ Parent: CollisionHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	6
nc	1
nop	2
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

<?php

namespace Ahc\Phint\Generator;

use Ahc\Phint\Util\Arr;
use Ahc\Phint\Util\Path;

class CollisionHandler implements CollisionHandlerInterface
{
    public function __construct(Path $pathUtil = null)
    {
        $this->pathUtil = $pathUtil ?: new Path;

    }

    /**
     * {@inheritdoc}
     */
    public function handle($targetFile, $newContent, array $parameters = null)
    {
        switch ($this->pathUtil->getExtension($targetFile)) {
            case 'json':
                $this->mergeJson($targetFile, $newContent);

                break;
        }
    }

    protected function mergeJson($targetFile, $newContent)
    {
        $oldJson = $this->pathUtil->readAsJson($targetFile);
        $newJson = \json_decode($newContent, true);
        $merged  = Arr::mergeRecursive($oldJson, $newJson);

        $this->pathUtil->writeFile(
            $targetFile,
            \json_encode($merged, \JSON_PRETTY_PRINT) . "\n"
        );
    }
}


1			<?php
2
3			namespace Ahc\Phint\Generator;
4
5			use Ahc\Phint\Util\Arr;
6			use Ahc\Phint\Util\Path;
7
8			class CollisionHandler implements CollisionHandlerInterface
9			{
10			public function __construct(Path $pathUtil = null)
11			{
12			$this->pathUtil = $pathUtil ?: new Path;
			0 ignored issues – show Bug Best Practice introduced 2017-10-29 10:46 UTC by Report Bug Copy Issue Report The property `pathUtil` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
13			}
14
15			/**
16			* {@inheritdoc}
17			*/
18			public function handle($targetFile, $newContent, array $parameters = null)
19			{
20			switch ($this->pathUtil->getExtension($targetFile)) {
21			case 'json':
22			$this->mergeJson($targetFile, $newContent);
23
24			break;
25			}
26			}
27
28			protected function mergeJson($targetFile, $newContent)
29			{
30			$oldJson = $this->pathUtil->readAsJson($targetFile);
31			$newJson = \json_decode($newContent, true);
32			$merged = Arr::mergeRecursive($oldJson, $newJson);
33
34			$this->pathUtil->writeFile(
35			$targetFile,
36			\json_encode($merged, \JSON_PRETTY_PRINT) . "\n"
37			);
38			}
39			}
40

adhocore / phint

Push — master ( dfd004...b46837 )

CollisionHandler::mergeJson() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like