Cp\Manager\PlanManager

Complexity

Total Complexity

4

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
B	findByType()	0	24	3

<?php

namespace Cp\Manager;

use Cp\DomainObject\Plan;
use Cp\Exception\ConfigurationException;
use Cp\Parser\PlanParser;
use Cp\Transformer\UrlTransformer;
use Doctrine\Common\Cache\MemcachedCache;
use JMS\Serializer\Serializer;

/**
 * Class PlanManager
 */
class PlanManager
{
    /**
     * @var PlanParser
     */
    private $planParser;

    /**
     * @var UrlTransformer
     */
    private $urlTransformer;

    /**
     * @var Serializer
     */
    private $serializer;

    /**
     * @var MemcachedCache
     */
    private $memcache;

    /**
     * TrainingManager constructor.
     *
     * @param PlanParser     $planParser
     * @param UrlTransformer $urlTransformer
     * @param Serializer     $serializer
     * @param MemcachedCache $memcached
     */
    public function __construct(
        PlanParser $planParser,
        UrlTransformer $urlTransformer,
        Serializer $serializer,
        MemcachedCache $memcached
    ) {
        $this->planParser = $planParser;

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        $this->urlTransformer = $urlTransformer;
        $this->serializer = $serializer;

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

        $this->memcache = $memcached;

Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space

    }

    /**
     * @param int    $week
     * @param int    $seance
     * @param string $type
     *
     * @return Plan
     * @throws ConfigurationException
     */
    public function findByType($week, $seance, $type)
    {
        $plan = $this->memcache->fetch($week.$seance.$type);
        if (false === $plan) {
            try {
                $jsonString = $this->planParser->parseToJson(
                    $this->urlTransformer->transformPlan($week, $seance, $type)
                );
            } catch (\Exception $e) {
                throw new ConfigurationException(
                    sprintf(
                        'Configuration with week: %s, seance: %s and type:%s is not available',
                        $week,
                        $seance,
                        $type
                    )
                );
            }
            $plan = $this->serializer->deserialize($jsonString, Plan::class, 'json');
            $this->memcache->save($week.$seance.$type, $plan, 3600);
        }

        return $plan;
    }
}

As per coding style, files should not end with a newline character.