NamespaceProtector\Config\ConfigTemplateCreator

Complexity

Total Complexity

3

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	createJsonTemplateConfig()	0	3	1
A	createJsonTemplateVisibility()	0	3	1
A	createFileWithBackup()	0	6	1

<?php
declare(strict_types=1);

namespace NamespaceProtector\Config;

use NamespaceProtector\Common\PathInterface;

final class ConfigTemplateCreator
{
    private const FILENAME = 'namespace-protector-config.json';
    private const FILENAME_VISIBILITY =  'namespace-protector-visibility.json';
    private const TEMPLATE_CONFIG_JSON = 'template-config-json';

    public static function createJsonTemplateConfig(PathInterface $baseComposerJsonDirectory): void
    {
        self::createFileWithBackup($baseComposerJsonDirectory().self::FILENAME, self::TEMPLATE_CONFIG_JSON);
    }

    public static function createJsonTemplateVisibility(): void
    {
        self::createFileWithBackup(self::FILENAME_VISIBILITY, 'template-visibility');
    }

    private static 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);
    }
}