Ray\Compiler\FilePutContents

Complexity

Total Complexity

5

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	13	5

<?php

declare(strict_types=1);

namespace Ray\Compiler;

use Ray\Compiler\Exception\FileNotWritable;

use function dirname;
use function file_put_contents;
use function is_dir;
use function is_string;
use function mkdir;
use function rename;
use function tempnam;
use function unlink;

final class FilePutContents
{
    public function __invoke(string $filename, string $content): void
    {
        $dir = dirname($filename);
        ! is_dir($dir) && mkdir($dir, 0777, true);
        $tmpFile = tempnam(dirname($filename), 'swap');
        if (is_string($tmpFile) && file_put_contents($tmpFile, $content) && @rename($tmpFile, $filename)) {
            return;
        }

        @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 FileNotWritable($filename);
    }
}