Easychimp\Easychimp

Complexity

Total Complexity

15

Size/Duplication

Total Lines	136
Duplicated Lines	22.06 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	4
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	isSubscribed()	15	15	3
A	__construct()	0	4	1
B	subscribe()	0	29	5
A	unsubscribe()	15	15	3
A	interestCategories()	0	6	1
A	interests()	0	6	1
A	hashEmail()	0	4	1

<?php

namespace Easychimp;

use Mailchimp\Mailchimp;

class Easychimp
{

    /** @var Mailchimp */
    protected $api;

    public function __construct($apiKey)
    {
        $this->api = new Mailchimp($apiKey);
    }

    /**
     * @param $listId
     * @param $email
     *
     * @throws \Exception
     *
     * @return boolean
     */
    public function isSubscribed($listId, $email)

This method seems to be duplicated in your project.

    {
        try {
            $result = $this->api->get('lists/'.$listId.'/members/'.$this->hashEmail($email));

            return $result->get('status') == 'subscribed';
        } catch (\Exception $e) {
            # Email address isn't on this list
            if (str_contains($e->getMessage(), 'Resource Not Found')) {
                return false;
            }

            throw $e;
        }
    }

    /**
     * @param $listId
     * @param $email
     * @param array $extras

There is no parameter named $extras. Was it maybe removed?

     *
     * @throws \Exception
     *
     * @return boolean
     */
    public function subscribe(
        $listId,
        $email,
        $firstName = null,
        $lastName = null,
        array $interests = null
    ) {
        $mergeFields = [];
        if (!is_null($firstName)) {

As per coding-style, please use === null instead of is_null.

            $mergeFields['FNAME'] = $firstName;
        }

        if (!is_null($lastName)) {

As per coding-style, please use === null instead of is_null.

            $mergeFields['LNAME'] = $lastName;
        }
        $data = [
            'email_address' => $email,
            'status'        => 'subscribed',
            'merge_fields'  => (object) $mergeFields
        ];

        if (!is_null($interests)) {

As per coding-style, please use === null instead of is_null.

            $data['interests'] = (object) array_flip($interests);
        }

        $result = $this->api->post('lists/'.$listId.'/members', $data);

        return $result->has('id') && strlen($result->get('id')) > 0;
    }

    /**
     * @param $listId
     * @param $email
     *
     * @throws \Exception
     *
     * @return boolean
     */
    public function unsubscribe($listId, $email)

This method seems to be duplicated in your project.

    {
        try {
            $result = $this->api->delete('lists/'.$listId.'/members/'.$this->hashEmail($email));

            return $result->count() == 0;
        } catch (\Exception $e) {
            # Email address isn't on this list
            if (str_contains($e->getMessage(), 'Resource Not Found')) {
                return true;
            }

            throw $e;
        }
    }

    /**
     * @param $listId
     *
     * @throws \Exception
     *
     * @return []

The doc-type [] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function interestCategories($listId)
    {
        $result = $this->api->get('lists/'.$listId.'/interest-categories');

        return $result->get('categories');
    }

    /**
     * @param $listId
     * @param $interestCategoryId
     *
     * @throws \Exception
     *
     * @return []

The doc-type [] could not be parsed: Unknown type name "" at position 0. [(view supported doc-types)

     */
    public function interests($listId, $interestCategoryId)
    {
        $result = $this->api->get('lists/'.$listId.'/interest-categories/'.$interestCategoryId.'/interests');

        return $result->get('interests');
    }

    /**
     * @param $email
     * @return string
     */
    protected function hashEmail($email)
    {
        return md5(strtolower($email));
    }
}