Cp\CapSniffer

Complexity

Total Complexity

6

Size/Duplication

Total Lines	109
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

88%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	1
A	generateCalendar()	0	4	1
A	writeCalendar()	0	7	1
A	getPlan()	0	8	1
A	getFileName()	0	4	1
A	testConfigurationParse()	0	5	1

<?php

namespace Cp;

use Cocur\Slugify\Slugify;
use Cp\Calendar\Builder\CalendarBuilder;
use Cp\DomainObject\Configuration;
use Cp\Manager\ConfigurationManager;
use Cp\Provider\PlanProvider;
use Cp\Provider\TypeProvider;

/**
 * Class CapSniffer
 */
class CapSniffer
{
    /**
     * @var TypeProvider
     */
    private $typeProvider;

    /**
     * @var PlanProvider
     */
    private $planProvider;

    /**
     * @var Slugify
     */
    private $slug;

    /**
     * @var CalendarBuilder
     */
    private $calendarBuilder;

    /**
     * @var ConfigurationManager
     */
    private $configurationManager;

    /**
     * CapSniffer constructor.
     *
     * @param TypeProvider         $typeProvider
     * @param PlanProvider         $planProvider
     * @param CalendarBuilder      $calendarBuilder
     * @param Slugify              $slug
     * @param ConfigurationManager $configurationManager
     */
    public function __construct(
        TypeProvider $typeProvider,
        PlanProvider $planProvider,
        CalendarBuilder $calendarBuilder,
        Slugify $slug,
        ConfigurationManager $configurationManager
    ) {
        $this->typeProvider = $typeProvider;
        $this->planProvider = $planProvider;
        $this->calendarBuilder = $calendarBuilder;
        $this->slug = $slug;
        $this->configurationManager = $configurationManager;
    }

    /**
     * @param string $typeName
     * @param string $week
     * @param string $seance
     *
     * @return string
     */
    public function generateCalendar($typeName, $week, $seance)
    {
        return $this->calendarBuilder->exportCalendar($this->getPlan($typeName, $week, $seance));
    }

    /**
     * @param string $typeName
     * @param string $week
     * @param string $seance
     */
    public function writeCalendar($typeName, $week, $seance)
    {
        file_put_contents(
            __DIR__.'/../../'.$this->getFileName($typeName, $week, $seance),
            $this->generateCalendar($typeName, $week, $seance)
        );
    }

    /**
     * @param string $typeName
     * @param string $week
     * @param string $seance
     *
     * @return Plan

Should the return type not be DomainObject\Plan?

     */
    public function getPlan($typeName, $week, $seance)
    {
        $configuration = $this->configurationManager->createConfiguration($typeName, $week, $seance);
        $plan = $this->planProvider->getPlanByConfiguration($configuration);
        $plan->setConfiguration($configuration);

        return $plan;
    }

    /**
     * @param string $typeName
     * @param string $week
     * @param string $seance
     *
     * @return string
     */
    public function getFileName($typeName, $week, $seance)
    {
        return $this->slug->slugify($this->getPlan($typeName, $week, $seance)->getName()).'.ics';
    }

    public function testConfigurationParse()
    {
        $result = $this->configurationManager->findConfigurationsByType('plan-entrainement-10km');
        var_dump($result); die;

var_dump($result); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

The method testConfigurationParse() contains an exit expression.

It is generally recommended to place each PHP statement on a line by itself.

    }
}