Ray\Aop\Code

Complexity

Total Complexity

5

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	5
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	save()	0	13	4
A	__construct()	0	3	1

<?php

declare(strict_types=1);

namespace Ray\Aop;

use PhpParser\Node;
use PhpParser\PrettyPrinter\Standard;
use Ray\Aop\Exception\NotWritableException;

use function dirname;
use function file_put_contents;
use function is_string;
use function rename;
use function sprintf;
use function str_replace;
use function tempnam;
use function unlink;

use const PHP_EOL;

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

    /** @param array<Node> $stmt */
    public function __construct(array $stmt)
    {
        $this->code = (new Standard(['shortArraySyntax' => true]))->prettyPrintFile($stmt) . PHP_EOL;
    }

    public function save(string $classDir, string $aopClassName): string
    {
        $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;
        }

        // @codeCoverageIgnoreStart
        @unlink((string) $tmpFile);

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

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

        // @codeCoverageIgnoreEnd
    }
}