Harvest

Complexity

Total Complexity

24

Size/Duplication

Total Lines	135
Duplicated Lines	48.15 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	1
A	startTimer()	8	21	3
A	stopTimer()	0	19	4
A	deleteTimer()	15	15	3
A	isTimerRunning()	0	14	4
A	getLastMessage()	0	4	1
A	setMessage()	0	4	1
A	sendApiCall()	17	17	3
A	lastApiCall()	14	14	4

<?php

require __DIR__ . '/../vendor/autoload.php';

use GuzzleHttp\Client;
use GuzzleHttp\Exception\ClientException;
use GuzzleHttp\Exception\ConnectException;

/**
 *
 */
class Harvest
{
    private $client = null;
    private $code = 0;
    private $message = '';
    private $data = null;

    public function __construct($domain = null, $apiToken = null)

This method seems to be duplicated in your project.

    {
        $this->client = new Client([
            'base_uri' => 'https://' . $domain . '.harvestapp.com/daily/',
            'headers' => [
                'Content-type' => 'application/json',
                'Accept' => 'application/json',
                'Authorization' => 'Basic ' . $apiToken,
            ],
        ]);
    }

    public function startTimer($description, $projectId, $taskId)
    {
        $harvestId = null;

        $item = [
            'notes' => $description,
            'project_id' => $projectId,
            'task_id' => $taskId,
        ];

        if ($this->sendApiCall('post', 'add', ['json' => $item]) === true) {

This code seems to be duplicated across your project.

            if ($this->lastApiCall('success') === true) {
                $this->setMessage('timer started');
                $harvestId = $this->data['id'];
            } else {
                $this->setMessage('cannot start timer!');
            }
        }

        return $harvestId;
    }

    public function stopTimer($timerId = null)
    {
        $res = false;

        if ($this->isTimerRunning($timerId) === true) {
            if ($this->sendApiCall('get', 'timer/' . $timerId) === true) {
                if ($this->lastApiCall('success') === true) {
                    $this->setMessage('timer stopped');
                    $res = true;
                } else {
                    $this->setMessage('could not stop timer!');
                }
            }
        } else {
            $this->setMessage('timer was not running');
        }

        return $res;
    }

    public function deleteTimer($timerId = null)

This method seems to be duplicated in your project.

    {
        $res = false;

        if ($this->sendApiCall('delete', 'delete/' . $timerId) === true) {
            if ($this->lastApiCall('success') === true) {
                $this->setMessage('timer deleted');
                $res = true;
            } else {
                $this->setMessage('could not delete timer!');
            }
        }

        return $res;
    }

    private function isTimerRunning($timerId)
    {
        $res = false;

        if ($this->sendApiCall('get', 'show/' . $timerId) === true) {
            if ($this->lastApiCall('success') === true) {
                if (isset($this->data['timer_started_at']) === true) {
                    $res = true;
                }
            }
        }

        return $res;
    }

    public function getLastMessage()
    {
        return $this->message;
    }

    private function setMessage($message = null)
    {
        $this->message = '- Harvest: ' . $message;
    }

    private function sendApiCall($method, $uri = '', array $options = [])

This method seems to be duplicated in your project.

    {
        $res = true;

        try {
            $response = $this->client->request(strtoupper($method), $uri, $options);
            $this->code = $response->getStatusCode();
            $this->data = json_decode($response->getBody(), true);
        } catch (ConnectException $e) {
            $this->setMessage('cannot connect to api!');
            $res = false;
        } catch (ClientException $e) {
            $this->setMessage($e->getResponse()->getBody());
        }

        return $res;
    }

    private function lastApiCall($status = '')

This method seems to be duplicated in your project.

    {
        $res = false;

        switch ($status) {
            case 'success':
                if ($this->code === 200 || $this->code === 201) {
                    $res = true;
                }
                break;
        }

        return $res;
    }
}