Fomvasss\Variable\VariableManager

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	locale()	0	5	1
A	__construct()	0	12	2
A	all()	0	14	3
A	get()	0	3	1

<?php

namespace Fomvasss\Variable;

/**
 * Class VariableManager
 *
 * @package \Fomvasss\Variable
 */
class VariableManager implements VariableManagerContract
{
    protected $variableModel;

    protected $cacheRepository;

    protected $locale;

    protected $variables;

    /**
     * VariableManager constructor.
     * @param $cacheRepository
     * @param null $app

Are you sure the doc-type for parameter $app is correct as it would always require null to be passed?

     */
    public function __construct($variableModel, $cacheRepository, $app = null)
    {
        if (! $app) {

$app is of type null, thus it always evaluated to false.

            $app = app();   //Fallback when $app is not given

The function app was not found. Maybe you did not declare it correctly or list all dependencies?

        }
        $this->app = $app;

The property app does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->config = $this->app['config']->get('variables');

The property config does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->variableModel = $variableModel;

        $this->cacheRepository = $cacheRepository;
    }

    /**
     * @param string $key
     * @param string|null $default
     * @return mixed|string
     */
    public function get(string $key, $default = null)
    {
        return $this->all()[$key] ?? $default;
    }

    /**
     * @return array
     */
    public function all(): array
    {
        if (! isset($this->variables)) {

            $all = $this->cacheRepository->remember($this->config['cache']['name'], $this->config['cache']['time'], function () {
                return $this->variableModel->select('key', 'value', 'locale')->get();
            });

            $allByLocale = $this->locale ? $all->where('locale', $this->locale) : $all;

            $this->variables = $allByLocale->pluck('value', 'key')->toArray();
        }

        return $this->variables;
    }

    /**
     * @param string $locale
     * @return $this
     */
    public function locale(string $locale = null)
    {
        $this->locale = $locale;

        return $this;
    }
}