hiqdev\composer\config\configs\Defines

Complexity

Total Complexity

3

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	loadFile()	0	11	2
A	writeFile()	0	3	1

<?php
/**
 * Composer plugin for config assembling
 *
 * @link      https://github.com/hiqdev/composer-config-plugin
 * @package   composer-config-plugin
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2016-2018, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\composer\config\configs;

/**
 * Defines class represents output configuration file with constant definitions.
 *
 * @author Andrii Vasyliev <sol@hiqdev.com>
 */
class Defines extends Config
{
    protected function loadFile($path)
    {
        parent::loadFile($path);
        if (pathinfo($path, PATHINFO_EXTENSION) !== 'php') {
            return [];
        }

        $lines = file($path);
        array_shift($lines);

It seems like $lines can also be of type false; however, parameter $array of array_shift() does only seem to accept array, maybe add an additional type check?

        return $lines;
    }

    protected function writeFile(string $path, array $data)
    {
        static::putFile($path, "<?php\n\n" . trim(implode('', $data)));
    }
}