Godbout\Alfred\Time\Services\Harvest

Complexity

Total Complexity

20

Size/Duplication

Total Lines	144
Duplicated Lines	11.11 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	projects()	0	8	2
A	tags()	0	17	2
A	pastTimers()	0	13	2
A	startTimer()	0	19	3
A	stopCurrentTimer()	16	16	3
A	runningTimer()	0	10	2
A	continueTimer()	0	6	1
A	deleteTimer()	0	10	2
A	convertToPastTimers()	0	6	1
A	buildPastTimerObject()	0	13	1

<?php

namespace Godbout\Alfred\Time\Services;

use Carbon\Carbon;
use Carbon\CarbonInterval;
use Required\Harvest\Client;
use Required\Harvest\Exception\NotFoundException;
use Required\Harvest\Exception\AuthenticationException;
use Required\Harvest\Exception\ValidationFailedException;

class Harvest extends TimerService
{
    public $allowsEmptyProject = false;

    public $allowsEmptyTag = false;

    private $client;


    public function __construct($accountId, $apiToken)
    {
        $this->client = new Client();

        $this->client->authenticate($accountId, $apiToken);
    }

    public function projects()
    {
        try {
            return array_column($this->client->projects()->all(['']), 'name', 'id');
        } catch (AuthenticationException $e) {
            return [];
        }
    }

    public function tags()
    {
        try {
            $taskAssignments = $this->client->projects()->taskAssignments()->all(
                (int) getenv('timer_project_id'),
                ['is_active' => true]
            );

            array_walk($taskAssignments, function ($taskAssignment) use (&$tags) {
                $tags[$taskAssignment['task']['id']] = $taskAssignment['task']['name'];
            });

            return $tags;
        } catch (AuthenticationException $e) {
            return [];
        }
    }

    public function pastTimers()
    {
        try {
            $harvestTimers = $this->client->timeEntries()->all([
                'from' => Carbon::today()->subDays(30),
                'to' => Carbon::today()
            ]);

            return $this->convertToPastTimers($harvestTimers);
        } catch (AuthenticationException $e) {
            return [];
        }
    }

    public function startTimer()
    {
        try {
            $timer = $this->client->timeEntries()->create([
                'notes' => getenv('timer_description'),
                'project_id' => (int) getenv('timer_project_id'),
                'task_id' => (int) getenv('timer_tag_id'),
                'spent_date' => date('Y-m-d')
            ]);

            if (! isset($timer['id'])) {
                return false;
            }
        } catch (ValidationFailedException $e) {
            return false;
        }

        return $timer['id'];
    }

    public function stopCurrentTimer()

This method seems to be duplicated in your project.

    {
        if ($timerId = $this->runningTimer()) {
            $timer = $this->client->timeEntries()->stop($timerId);

            if (! isset($timer['id'])) {
                throw new \Exception("Can't stop current running timer.", 1);

                return false;

return false; does not seem to be reachable.

            }

            return true;
        }

        return false;
    }

    public function runningTimer()
    {
        try {
            $timer = $this->client->timeEntries()->all(['is_running' => true]);

            return $timer[0]['id'] ?? false;
        } catch (\Exception $e) {
            return false;
        }
    }

    public function continueTimer($timerId)
    {
        $timer = $this->client->timeEntries()->restart($timerId);

        return $timer['id'] ?? false;
    }

    public function deleteTimer($timerId)
    {
        try {
            $this->client->timeEntries()->remove($timerId);
        } catch (NotFoundException $e) {
            return false;
        }

        return true;
    }

    protected function convertToPastTimers($harvestTimers)
    {
        return array_map(function ($harvestTimer) {
            return $this->buildPastTimerObject($harvestTimer);
        }, $harvestTimers);
    }

    protected function buildPastTimerObject($harvestTimer)
    {
        $pastTimer['id'] = $harvestTimer['id'];

$pastTimer was never initialized. Although not strictly required by PHP, it is generally a good practice to add $pastTimer = array(); before regardless.

        $pastTimer['description'] = $harvestTimer['notes'];
        $pastTimer['project_id'] = $harvestTimer['project']['id'];
        $pastTimer['project_name'] = $harvestTimer['project']['name'];
        $pastTimer['tags'] = $harvestTimer['task']['name'];
        $pastTimer['duration'] = CarbonInterval::seconds(
            floor($harvestTimer['hours'] * 3600)
        )->cascade()->format('%H:%I:%S');

        return (object) $pastTimer;
    }
}