AlfredTime\Toggl

Complexity

Total Complexity

17

Size/Duplication

Total Lines	166
Duplicated Lines	28.31 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	11	11	1
A	deleteTimer()	12	12	2
A	getLastMessage()	0	4	1
A	getOnlineData()	12	12	2
A	getProjects()	0	4	1
A	getRecentTimers()	0	4	1
B	startTimer()	0	24	2
A	stopTimer()	12	12	2
A	setMessage()	0	4	1
B	timerAction()	0	26	4

<?php

namespace AlfredTime;

use AlfredTime\ServiceApiCall;

/**
 *
 */
class Toggl
{
    /**
     * @var string
     */
    private $message = '';

    /**
     * @var mixed
     */
    private $serviceApiCall = null;

    /**
     * @param $apiToken
     */
    public function __construct($apiToken = null)

This method seems to be duplicated in your project.

    {
        $this->serviceApiCall = new ServiceApiCall([
            'base_uri' => 'https://www.toggl.com/api/v8/',
            'headers'  => [
                'Content-type'  => 'application/json',
                'Accept'        => 'application/json',
                'Authorization' => 'Basic ' . base64_encode($apiToken . ':api_token'),
            ],
        ]);
    }

    /**
     * @param  $timerId
     * @return mixed
     */
    public function deleteTimer($timerId = null)

This method seems to be duplicated in your project.

    {
        $res = $this->timerAction('delete', 'time_entries/' . $timerId);

        if ($res === true) {
            $this->setMessage('timer deleted');
        } else {
            $this->setMessage('could not delete timer! [' . $this->message . ']');
        }

        return $res;
    }

    /**
     * @return string
     */
    public function getLastMessage()
    {
        return $this->message;
    }

    /**
     * @return mixed
     */
    public function getOnlineData()

This method seems to be duplicated in your project.

    {
        $data = $this->timerAction('get_online_data', 'me?with_related_data=true');

        if (empty($data) === false) {
            $this->setMessage('data cached');
        } else {
            $this->setMessage('cannot get online data! [' . $this->message . ']');
        }

        return $data;
    }

    public function getProjects()
    {
        # code...
    }

    public function getRecentTimers()
    {
        return array_reverse($this->timerAction('get_recent_timers', 'time_entries'));
    }

    /**
     * @param  $description
     * @param  $projectId
     * @param  $tagNames
     * @return mixed
     */
    public function startTimer($description, $projectId, $tagNames)
    {
        $togglId = null;

        $item = [
            'time_entry' => [
                'description'  => $description,
                'pid'          => $projectId,
                'tags'         => explode(', ', $tagNames),
                'created_with' => 'Alfred Time Workflow',
            ],
        ];

        $data = $this->timerAction('start', 'time_entries/start', ['json' => $item]);

        if (isset($data['data']['id']) === true) {
            $this->setMessage('timer started');
            $togglId = $data['data']['id'];
        } else {
            $this->setMessage('could not start timer! [' . $this->message . ']');
        }

        return $togglId;
    }

    /**
     * @param  $timerId
     * @return mixed
     */
    public function stopTimer($timerId = null)

This method seems to be duplicated in your project.

    {
        $res = $this->timerAction('stop', 'time_entries/' . $timerId . '/stop');

        if ($res === true) {
            $this->setMessage('timer stopped');
        } else {
            $this->setMessage('could not stop timer! [' . $this->message . ']');
        }

        return $res;
    }

    /**
     * @param string $message
     */
    private function setMessage($message = null)
    {
        $this->message = '- Toggl: ' . $message;
    }

    /**
     * @param  string  $action
     * @param  string  $apiUri
     * @return mixed
     */
    private function timerAction($action, $apiUri, array $options = [])
    {
        $res = false;
        $method = '';

$method is not used, you could remove the assignment.

        $returnDataFor = ['start', 'get_recent_timers', 'get_online_data'];
        $methods = [
            'start'             => 'post',
            'stop'              => 'put',
            'delete'            => 'delete',
            'get_recent_timers' => 'get',
            'get_online_data'   => 'get',
        ];
        $method = isset($methods[$action]) ? $methods[$action] : '';

        if ($this->serviceApiCall->send($method, $apiUri, $options) === true) {
            $res = $this->serviceApiCall->last('success');

            if (in_array($action, $returnDataFor) === true) {
                $res = $this->serviceApiCall->getData();
            }
        } else {
            $this->message = $this->serviceApiCall->getMessage();
        }

        return $res;
    }
}