DP\Core\CoreBundle\Settings\SettingsReader

Complexity

Total Complexity

5

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	read()	0	12	2
A	fileExists()	0	4	1
A	isReadable()	0	4	1

<?php

namespace DP\Core\CoreBundle\Settings;

use Psr\Log\LoggerInterface;
use Symfony\Component\Yaml\Exception\ParseException;
use Symfony\Component\Yaml\Yaml;

class SettingsReader
{
    /**
     * @var string
     */
    private $filePath;

    /**
     * @var Logger
     */
    private $logger;

    /**
     * @param string $filePath
     * @param LoggerInterface $logger
     */
    public function __construct($filePath, LoggerInterface $logger)
    {
        $this->filePath = $filePath;
        $this->logger   = $logger;

It seems like $logger of type object<Psr\Log\LoggerInterface> is incompatible with the declared type object<DP\Core\CoreBundle\Settings\Logger> of property $logger.

    }

    /**
     * @return array
     */
    public function read()
    {
        $settings = [];

        try {
            $settings = Yaml::parse($this->filePath);
        } catch (ParseException $e) {
            $this->logger->error(sprintf('Unable to read the core settings file "%s".', $this->filePath));
        }

        return $settings;
    }

    /**
     * @return bool
     */
    public function fileExists()
    {
        return file_exists($this->filePath);
    }

    /**
     * @return bool
     */
    public function isReadable()
    {
        return is_readable($this->filePath);
    }
}