TrelloBurndown\Manager\ListManager

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	15.63 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getListFromBoard()	5	12	3
B	getList()	5	19	5

<?php

namespace TrelloBurndown\Manager;

use Trello\Model\Board;
use TrelloBurndown\Client\TrelloClient;
use Trello\Model\Cardlist;

/**
 * Class ListManager.
 */
class ListManager
{
    /**
     * @var \Trello\Client
     */
    private $client;

    /**
     * ListManager constructor.
     *
     * @param TrelloClient $client
     */
    public function __construct(TrelloClient $client)
    {
        $this->client = $client->getClient();
    }

    /**
     * @param string $listName
     * @param Board  $board
     *
     * @return Cardlist|void
     */
    public function getListFromBoard(String $listName, Board $board)
    {
        $lists = $this->client->api('board')->lists()->all($board->getId());

        foreach ($lists as $list) {

This code seems to be duplicated across your project.

            if ($list['name'] == $listName) {
                return new Cardlist($this->client, $list['id']);
            }
        }

        return;
    }

    /**
     * @param string $listName
     * @param array  $boards
     *
     * @return Cardlist|void
     *
     * @throws \Exception
     */
    public function getList(String $listName, array $boards)
    {
        $lists = [];
        foreach ($boards as $board) {
            if (!($board instanceof Board)) {
                throw new \Exception('Function ListManager::getList expect an array of Board as second argument');
            }

            $lists = array_merge($lists, $this->client->api('board')->lists()->all($board->getId()));
        }

        foreach ($lists as $list) {

This code seems to be duplicated across your project.

            if ($list['name'] == $listName) {
                return new Cardlist($this->client, $list['id']);
            }
        }

        return;
    }
}