PhpMob\Settings\Provider\CompositeSettingProvider

Complexity

Total Complexity

7

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	findUserSettings()	0	7	1
A	findGlobalSettings()	0	7	1
A	mergeSettings()	0	13	3
A	exists()	0	6	1

<?php

/*
 * This file is part of the PhpMob package.
 *
 * (c) Ishmael Doss <nukboon@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpMob\Settings\Provider;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use PhpMob\Settings\Model\SettingInterface;

/**
 * @author Ishmael Doss <nukboon@gmail.com>
 */
class CompositeSettingProvider implements SettingProviderInterface
{
    /**
     * @var SettingProviderInterface
     */
    private $localProvider;

    /**
     * @var SettingProviderInterface
     */
    private $remoteProvider;

    /**
     * @param SettingProviderInterface $localProvider
     * @param SettingProviderInterface $remoteProvider
     */
    public function __construct(SettingProviderInterface $localProvider, SettingProviderInterface $remoteProvider)
    {
        $this->localProvider = $localProvider;
        $this->remoteProvider = $remoteProvider;
    }

    /**
     * {@inheritdoc}
     */
    public function findUserSettings(string $owner): Collection
    {
        return $this->mergeSettings(
            $this->remoteProvider->findUserSettings($owner),

It seems like $this->remoteProvider->findUserSettings($owner) targeting PhpMob\Settings\Provider\SettingProviderInterface::findUserSettings() can also be of type array<integer,object<PhpMob\Settings\Model\SettingInterface>>; however, PhpMob\Settings\Provider\CompositeSettingProvider::mergeSettings() does only seem to accept object<Doctrine\Common\Collections\Collection>, maybe add an additional type check?

            $this->localProvider->findUserSettings($owner)

It seems like $this->localProvider->findUserSettings($owner) targeting PhpMob\Settings\Provider\SettingProviderInterface::findUserSettings() can also be of type array<integer,object<PhpMob\Settings\Model\SettingInterface>>; however, PhpMob\Settings\Provider\CompositeSettingProvider::mergeSettings() does only seem to accept object<Doctrine\Common\Collections\Collection>, maybe add an additional type check?

        );
    }

    /**
     * {@inheritdoc}
     */
    public function findGlobalSettings(): Collection
    {
        return $this->mergeSettings(
            $this->remoteProvider->findGlobalSettings(),

It seems like $this->remoteProvider->findGlobalSettings() targeting PhpMob\Settings\Provider\SettingProviderInterface::findGlobalSettings() can also be of type array<integer,object<PhpMob\Settings\Model\SettingInterface>>; however, PhpMob\Settings\Provider\CompositeSettingProvider::mergeSettings() does only seem to accept object<Doctrine\Common\Collections\Collection>, maybe add an additional type check?

            $this->localProvider->findGlobalSettings()

It seems like $this->localProvider->findGlobalSettings() targeting PhpMob\Settings\Provider\SettingProviderInterface::findGlobalSettings() can also be of type array<integer,object<PhpMob\Settings\Model\SettingInterface>>; however, PhpMob\Settings\Provider\CompositeSettingProvider::mergeSettings() does only seem to accept object<Doctrine\Common\Collections\Collection>, maybe add an additional type check?

        );
    }

    /**
     * @param Collection $remotes
     * @param Collection $locals
     *
     * @return ArrayCollection
     */
    private function mergeSettings(Collection $remotes, Collection $locals): ArrayCollection
    {
        $settings = $remotes->toArray();

        /** @var SettingInterface $local */
        foreach ($locals->toArray() as $local) {
            if (!$this->exists($remotes, $local)) {
                $settings[] = $local;
            }
        }

        return new ArrayCollection($settings);
    }

    /**
     * @param Collection $remote
     * @param SettingInterface $object
     *
     * @return bool
     */
    private function exists(Collection $remote, SettingInterface $object): bool
    {
        return $remote->exists(function (int $key, SettingInterface $setting) use ($object) {
            return $setting->isEqual($object);
        });
    }
}