NamespaceProtector\Config\ConfigTemplateCreator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	3	1
A	__construct()	0	3	1
A	createFileWithBackup()	0	6	1

<?php
declare(strict_types=1);

namespace NamespaceProtector\Config;

use NamespaceProtector\Common\PathInterface;

final class ConfigTemplateCreator implements ConfigTemplateCreatorInterface
{
    /** @var string */
    private $templateName;

    public function __construct(string $templateName)
    {
        $this->templateName = $templateName;
    }

    public function create(PathInterface $destinationPathFileJson): void
    {
        $this->createFileWithBackup($destinationPathFileJson->get(), $this->templateName);
    }

    private function createFileWithBackup(string $fileName, string $templateFile): void
    {
        @\rename($fileName, $fileName . '_backup.json');

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

        $content = \safe\file_get_contents(__DIR__ . '/' . $templateFile);
        \safe\file_put_contents($fileName, $content);
    }
}