ONGR\SettingsBundle\Settings\General\SettingsManager

Complexity

Total Complexity

13

Size/Duplication

Total Lines	166
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	5
Bugs	0	Features	2

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	set()	0	18	2
A	save()	0	9	2
A	remove()	0	8	1
A	duplicate()	0	13	2
A	get()	0	12	3
A	createSetting()	0	14	2

<?php

/*
 * This file is part of the ONGR package.
 *
 * (c) NFQ Technologies UAB <info@nfq.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ONGR\SettingsBundle\Settings\General;

use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symfony\Component\Translation\TranslatorInterface;
use ONGR\ElasticsearchBundle\Service\Repository;
use ONGR\ElasticsearchBundle\Service\Manager;
use ONGR\SettingsBundle\Document\Setting;
use ONGR\SettingsBundle\Event\SettingChangeEvent;

/**
 * Class SettingsManager responsible for managing settings actions.
 */
class SettingsManager
{
    /**
     * @var TranslatorInterface
     */
    protected $translator;

    /**
     * @var EventDispatcherInterface
     */
    protected $eventDispatcher;

    /**
     * @var Manager
     */
    protected $manager;

    /**
     * @var Repository
     */
    protected $repo;

    /**
     * @param TranslatorInterface      $translator
     * @param EventDispatcherInterface $eventDispatcher
     * @param Manager                  $manager
     */
    public function __construct(
        TranslatorInterface $translator,
        EventDispatcherInterface $eventDispatcher,
        Manager $manager
    ) {
        $this->translator = $translator;
        $this->eventDispatcher = $eventDispatcher;
        $this->manager = $manager;
        $this->repo = $this->manager->getRepository('ONGRSettingsBundle:Setting');
    }

    /**
     * Overwrites setting with given name.
     *
     * @param string       $name
     * @param string       $type
     * @param string       $description
     * @param string|array $value
     * @param array        $profiles
     *
     * @throws \LogicException
     */
    public function set($name, $type, $description, $value, $profiles)
    {
        foreach ($profiles as $profile) {
            $setting = new Setting();
            $setting->setId($profile . '_' . $name);
            $setting->setName($name);
            $setting->setDescription($description);
            $setting->setData((object)['value' => $value]);
            $setting->setType($type);
            $setting->setProfile($profile);

            $this->manager->persist($setting);
        }
        $this->manager->commit();
        $this->manager->flush();

        $this->eventDispatcher->dispatch('ongr_settings.setting_change', new SettingChangeEvent('save'));

'save' is of type string, but the function expects a object<ONGR\SettingsBundle\Event\Action>.

    }

    /**
     * Saves setting.
     *
     * @param array $settings
     */
    public function save(array $settings)
    {
        foreach ($settings as $setting) {
            $this->manager->persist($setting);
        }
        $this->manager->commit();

        $this->eventDispatcher->dispatch('ongr_settings.setting_change', new SettingChangeEvent('save'));

'save' is of type string, but the function expects a object<ONGR\SettingsBundle\Event\Action>.

    }

    /**
     * Removes a setting.
     *
     * @param Setting $setting
     */
    public function remove(Setting $setting)
    {
        $this->repo->remove($setting->getId());
        $this->manager->flush();
        $this->manager->refresh();

        $this->eventDispatcher->dispatch('ongr_settings.setting_change', new SettingChangeEvent('delete'));

'delete' is of type string, but the function expects a object<ONGR\SettingsBundle\Event\Action>.

    }

    /**
     * Copy a setting to the new profile.
     *
     * @param Setting $setting
     * @param array  $newProfiles
     */
    public function duplicate(Setting $setting, $newProfiles)
    {
        $newSettings = [];
        foreach ($newProfiles as $profile) {
            $newSetting = clone $setting;

            $newSetting->setId($profile . '_' . $setting->getName());
            $newSetting->setProfile($profile);
            $newSettings[] = $newSetting;
        }

        $this->save($newSettings);
    }

    /**
     * Returns setting model by name and profile or creates new if $mustExist is set to FALSE.
     *
     * @param string $name
     * @param string $profile
     * @param bool   $mustExist
     * @param string $type
     *
     * @throws \UnexpectedValueException
     *
     * @return Setting
     */
    public function get($name, $profile = 'default', $mustExist = true, $type = 'string')
    {
        $setting = $this->repo->find($profile . '_' . $name);

The expression $this->repo->find($profile . '_' . $name); of type null|ReflectionClass adds the type ReflectionClass to the return on line 163 which is incompatible with the return type documented by ONGR\SettingsBundle\Settings\General\SettingsManager::get of type ONGR\SettingsBundle\Document\Setting.

        if ($setting === null) {
            if ($mustExist === true) {
                throw new \UnexpectedValueException();
            }
            $setting = $this->createSetting($name, $profile, $type);
        }

        return $setting;
    }

    /**
     * Creates new setting object.
     *
     * @param string $name
     * @param string $profile
     * @param string $type
     *
     * @return Setting
     */
    protected function createSetting($name, $profile, $type)
    {
        $setting = new Setting();
        $setting->setId($profile . '_' . $name);
        $setting->setName($name);
        $setting->setProfile($profile);
        $setting->setType($type);

        if ($type == 'array') {
            $setting->setData(['value' => null]);

array('value' => null) is of type array<string,null,{"value":"null"}>, but the function expects a string.

        }

        return $setting;
    }
}