TestMonitor\DoneDone\Actions\ManagesProjects

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

88.89%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	projects()	0	12	2
A	project()	0	6	1

<?php

namespace TestMonitor\DoneDone\Actions;

use TestMonitor\DoneDone\Resources\Project;
use TestMonitor\DoneDone\Transforms\TransformsProjects;

trait ManagesProjects
{
    use TransformsProjects;

    /**
     * Get a list of of projects.
     *
     * @param int $accountId
     * @return Project[]
     */
    public function projects(int $accountId): array
    {
        $result = $this->get("{$accountId}/internal-projects");

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($result === '[]') {
            return [];
        }

        return array_map(function ($project) {
            return $this->fromDoneDoneProject($project);
        }, $result);
    }

    /**
     * Get a single project.
     *
     * @param int $accountId
     * @param int $id
     *
     * @return Project
     */
    public function project(int $accountId, int $id): Project
    {
        $result = $this->get("{$accountId}/internal-projects/{$id}");

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->fromDoneDoneProject($result);
    }
}