Scrumban\Manager\BoardManager

Complexity

Total Complexity

17

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	sync()	0	19	5
A	init()	0	3	1
A	__construct()	0	6	1
A	getCardEstimations()	0	12	4
B	processColumnCards()	0	26	6

<?php

namespace Scrumban\Manager;

use Scrumban\Registry\RegistryInterface;
use Scrumban\Gateway\GatewayInterface;

use Scrumban\Utils\CardHelper;

use Scrumban\Utils\PlusForTrelloHelper;

use Scrumban\Entity\Sprint;

class BoardManager
{
    /** @var RegistryInterface **/
    protected $registry;
    /** @var GatewayInterface **/
    protected $gateway;
    /** @var UserStoryManager **/
    protected $userStoryManager;
    /** @var SprintManager **/
    protected $sprintManager;
    /** @var bool **/
    public $hasPlusForTrello;
    
    public function __construct(RegistryInterface $registry, GatewayInterface $gateway, UserStoryManager $userStoryManager, SprintManager $sprintManager)
    {
        $this->registry = $registry;
        $this->gateway = $gateway;
        $this->userStoryManager = $userStoryManager;
        $this->sprintManager = $sprintManager;
    }
    
    public function sync($boardName)
    {
        if (($boardData = $this->registry->getBoard($boardName)) === null) {
            throw new \InvalidArgumentException('No board is configured for the given name');
        }
        $this->init();
        $columns = $this->gateway->getBoardColumns($boardData['id']);

The method getBoardColumns() does not exist on Scrumban\Gateway\GatewayInterface. It seems like you code against a sub-type of Scrumban\Gateway\GatewayInterface such as Scrumban\Gateway\TrelloGateway.

        $currentSprint = $this->sprintManager->getCurrentSprint();

        foreach ($columns as $columnData) {
            yield "comment" => "Scanning column {$columnData['name']}...";
            if (($column = $this->registry->getColumn(CardHelper::slugify($columnData['name']))) === null) {
                continue;
            }
            $messages = $this->processColumnCards($columnData['id'], $column, $currentSprint);
            foreach($messages as $type => $message) {
                yield $type => $message;
            }
            yield "info" => "Column {$columnData['name']} has been synchronized";
        }
    }
    
    protected function init()
    {
        $this->registry->storeUserStories($this->userStoryManager->getAll());

The method storeUserStories() does not exist on Scrumban\Registry\RegistryInterface. Since it exists in all sub-types, consider adding an abstract or default implementation to Scrumban\Registry\RegistryInterface.

    }
    
    protected function processColumnCards(string $columnId, array $column, Sprint $sprint = null)
    {
        $cards = $this->gateway->getColumnCards($columnId);

The method getColumnCards() does not exist on Scrumban\Gateway\GatewayInterface. It seems like you code against a sub-type of Scrumban\Gateway\GatewayInterface such as Scrumban\Gateway\TrelloGateway.

        yield "info" => count($cards) . " cards to synchronize with status {$column['status']}";
        foreach ($cards as $card) {
            $titleParts = explode('|', $card['name']);
            $extraData = isset($titleParts[1]) ? trim($titleParts[1]): '';
            
            $estimations = $this->getCardEstimations($card, $extraData);
                    
            $method =
                (($userStory = $this->registry->getUserStory($card['id'])) === null)

The method getUserStory() does not exist on Scrumban\Registry\RegistryInterface. Since it exists in all sub-types, consider adding an abstract or default implementation to Scrumban\Registry\RegistryInterface.

                ? 'createUserStory'
                : 'updateUserStory'
            ;
            $this->userStoryManager->{$method}(
                $card['id'],
                trim($titleParts[0]),
                $card['desc'],
                (!empty($extraData)) ? CardHelper::extractValue($extraData) : 0,
                $column['status'],
                $estimations['estimated'],
                $estimations['spent'],
                CardHelper::extractCreationDate($card['id']),
                CardHelper::isInCurrentSprint($column['status']) ? $sprint : null,
                $userStory
            );
        }
    }
    
    protected function getCardEstimations(array $card, string $extraData): array
    {
        if ($this->hasPlusForTrello !== true || empty($extraData)) {
            return [
                'estimated' => CardHelper::extractEstimatedTime($extraData),
                'spent' => CardHelper::extractSpentTime($extraData)
            ];
        }
        if ($card['badges']['comments'] === 0) {
            return ['estimated' => 0, 'spent' => 0];
        }
        return PlusForTrelloHelper::extractEstimations($this->gateway->getCardComments($card['id']));

The method getCardComments() does not exist on Scrumban\Gateway\GatewayInterface. It seems like you code against a sub-type of Scrumban\Gateway\GatewayInterface such as Scrumban\Gateway\TrelloGateway.

    }
}