Ray\Aop\Code

Complexity

Total Complexity

5

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

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

<?php

declare(strict_types=1);

namespace Ray\Aop;

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

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
    {
        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);

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

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