Code::save() - Code Metrics - Inspection of "Safer file_put_contents" - ray-di/Ray.Aop - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 2.x (#137)

by Akihito

created 2020-05-13 05:16 UTC

Code::save() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.9666
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

declare(strict_types=1);

namespace Ray\Aop;

use Ray\Aop\Exception\NotWritableException;

final class Code
{
    /**
     * @var string
     */
    public $code = '';

s    public function save(string $classDir, string $aopClassName) : string

    {
        class_exists($aopClassName);
        $flatName = str_replace('\\', '_', $aopClassName);
        $filename = sprintf('%s/%s.php', $classDir, $flatName);
        $tmpFile = tempnam(dirname($filename), 'swap');
        if (is_string($tmpFile) && file_put_contents($tmpFile, $this->code) && rename($tmpFile, $filename)) {
            return $filename;
        }
        @unlink((string) $tmpFile);

        throw new NotWritableException(sprintf('swap: %s, file: %s', $tmpFile, $filename));
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Ray\Aop;
6
7			use Ray\Aop\Exception\NotWritableException;
8
9			final class Code
10			{
11			/**
12			* @var string
13			*/
14			public $code = '';
15
16			s public function save(string $classDir, string $aopClassName) : string
			0 ignored issues – show Bug introduced 2020-05-13 05:22 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Syntax error, unexpected T_STRING, expecting T_FUNCTION or T_CONST Loading history...
17			{
18			class_exists($aopClassName);
19			$flatName = str_replace('\\', '_', $aopClassName);
20			$filename = sprintf('%s/%s.php', $classDir, $flatName);
21			$tmpFile = tempnam(dirname($filename), 'swap');
22			if (is_string($tmpFile) && file_put_contents($tmpFile, $this->code) && rename($tmpFile, $filename)) {
23			return $filename;
24			}
25			@unlink((string) $tmpFile);
26
27			throw new NotWritableException(sprintf('swap: %s, file: %s', $tmpFile, $filename));
28			}
29			}
30

ray-di / Ray.Aop

Pull Request — 2.x (#137)

Code::save() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like