Mrkj\Laposta\Laposta

Complexity

Total Complexity

13

Size/Duplication

Total Lines	109
Duplicated Lines	22.02 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	1	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	18	4
A	getLists()	0	12	2
A	getList()	0	8	1
A	createList()	13	13	1
A	updateList()	11	11	1
A	get()	0	4	1
A	post()	0	4	1
A	request()	0	12	2

<?php

namespace Mrkj\Laposta;

use GuzzleHttp\Client;
use Mrkj\Laposta\Models\List_;
use GuzzleHttp\Exception\RequestException;

class Laposta
{
    /**
     * @var string
     */
    private $apiKey;

    /**
     * @var Client
     */
    private $client;

    public function __construct($apiKey = null, $client = null)
    {
        if (empty(trim($apiKey))) {
            throw new \Exception('No API key provided');
        }

        $this->apiKey = $apiKey;

        if ($client && $client instanceof Client) {
            $this->client = $client;
        } else {
            $this->client = new Client([
                'base_uri' => 'https://api.laposta.nl/v2/',
                'auth' => [$this->apiKey, ''],
                'http_errors' => true,
            ]);
        }
    }

    /**
     * @return List_[]
     */
    public function getLists()
    {
        $data = $this->get('list');

        $lists = [];

        foreach ($data['data'] as $listResponse) {
            $lists[] = List_::createFromResponse($listResponse['list']);
        }

        return $lists;
    }

    /**
     * @param $listId
     * @return bool|List_
     */
    public function getList($listId)
    {
        $data = $this->get('list/'.$listId);

        $list = List_::createFromResponse($data['list']);

        return $list;
    }

    public function createList(List_ $list)

This method seems to be duplicated in your project.

    {
        $data = $this->post('list', [
            'form_params' => [
                'name' => $list->getName(),
                'remarks' => $list->getRemarks(),
                'subscribe_notification_email' => $list->getSubscribeNotificationEmail(),
                'unsubscribe_notification_email' => $list->getUnsubsribeNotificationEmail(),
            ],
        ]);

        $list->updateFromResponse($data['list']);
    }

    public function updateList(List_ $list)

This method seems to be duplicated in your project.

    {
        $this->client->post('list/'.$list->getListId(), [
            'form_params' => [
                'name' => $list->getName(),
                'remarks' => $list->getRemarks(),
                'subscribe_notification_email' => $list->getSubscribeNotificationEmail(),
                'unsubscribe_notification_email' => $list->getUnsubsribeNotificationEmail(),
            ],
        ]);
    }

    private function get($uri, $options = [])
    {
        return $this->request('get', $uri, $options);
    }

    private function post($uri, $options = [])
    {
        return $this->request('post', $uri, $options);
    }

    private function request($method, $uri, $options = [])
    {
        try {
            $response = $this->client->request($method, $uri, $options);

            $data = json_decode($response->getBody()->getContents(), true);

            return $data;
        } catch (RequestException $e) {
            throw new \Exception('Loading Laposta resource failed: '.$e->getMessage());
        }
    }
}