TestMonitor\ActiveCampaign\Actions\ManagesContacts

Complexity

Total Complexity

19

Size/Duplication

Total Lines	229
Duplicated Lines	12.23 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	contacts()	0	8	1
A	findContact()	0	10	1
A	createContact()	0	9	1
A	findOrCreateContact()	0	10	2
A	contactAutomations()	0	8	1
A	contactTags()	0	8	1
A	removeAutomationFromContact()	14	14	2
A	removeAllAutomationsFromContact()	0	8	2
A	removeTagFromContact()	14	14	2
A	findContactById()	0	10	1
A	updateContact()	0	8	1
A	deleteContact()	0	4	1
A	getContactId()	0	12	3

<?php

namespace TestMonitor\ActiveCampaign\Actions;

use TestMonitor\ActiveCampaign\Resources\Tag;
use TestMonitor\ActiveCampaign\Resources\Contact;
use TestMonitor\ActiveCampaign\Resources\Automation;
use TestMonitor\ActiveCampaign\Resources\ContactTag;
use TestMonitor\ActiveCampaign\Resources\ContactAutomation;

trait ManagesContacts
{
    /**
     * Get all contacts.
     *
     * @return array
     */
    public function contacts()
    {
        return $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get('contacts'),

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            Contact::class,
            'contacts'
        );
    }

    /**
     * Find contact by email.
     *
     * @param string $email
     *
     * @return Contact|null
     */
    public function findContact($email)
    {
        $contacts = $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get('contacts', ['query' => ['email' => $email]]),

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            Contact::class,
            'contacts'
        );

        return array_shift($contacts);
    }

    /**
     * Find contact by ID.
     *
     * @param int $id
     *
     * @return Contact|null
     */
    public function findContactById($id)
    {
        $contacts = $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get('contacts', ['query' => ['id' => $id]]),

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            Contact::class,
            'contacts'
        );

        return array_shift($contacts);
    }

    /**
     * Create new contact.
     *
     * @param string $email
     * @param string $firstName
     * @param string $lastName
     * @param int|null $orgid
     *
     * @return Contact|null
     */
    public function createContact($email, $firstName, $lastName, $orgid = null)
    {
        $contacts = $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->post('contacts', ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]),

It seems like post() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            Contact::class
        );

        return array_shift($contacts);
    }

    /**
     * Find or create a contact.
     *
     * @param string $email
     * @param string $firstName
     * @param string $lastName
     * @param int|null $orgid
     *
     * @return Contact
     */
    public function findOrCreateContact($email, $firstName, $lastName, $orgid = null)
    {
        $contact = $this->findContact($email);

        if ($contact instanceof Contact) {
            return $contact;
        }

        return $this->createContact($email, $firstName, $lastName, $orgid);
    }

    /**
     * Updates a contact.
     *
     * @param Contact|int|string $id
     * @param string|null        $email
     * @param string|null        $firstName
     * @param string|null        $lastName
     * @param null               $orgid
     *
     * @return Contact|null
     */
    public function updateContact($id, $email, $firstName, $lastName, $orgid = null)
    {
        $id = $this->getContactId($id);

        $this->put('contacts/'.$id, ['json' => ['contact' => compact('email', 'firstName', 'lastName', 'orgid')]]);

It seems like put() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->findContactById($id);
    }

    /**
     * Deletes a contact.
     *
     * @param Contact|int|string $contact
     */
    public function deleteContact($contact)
    {
        $this->delete('contacts/'.$this->getContactId($contact));

The method delete() does not exist on TestMonitor\ActiveCampaign\Actions\ManagesContacts. Did you maybe mean deleteContact()?

    }

    /**
     * Get all automations of a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     *
     * @return array
     */
    public function contactAutomations(Contact $contact)
    {
        return $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get("contacts/{$contact->id}/contactAutomations"),

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ContactAutomation::class,
            'contactAutomations'
        );
    }

    /**
     * Get all tags of a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     *
     * @return array
     */
    public function contactTags(Contact $contact)
    {
        return $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get("contacts/{$contact->id}/contactTags"),

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ContactTag::class,
            'contactTags'
        );
    }

    /**
     * Removing a automation from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     * @param \TestMonitor\ActiveCampaign\Resources\Automation $automation
     */
    public function removeAutomationFromContact(Contact $contact, Automation $automation)

This method seems to be duplicated in your project.

    {
        $contactAutomations = $this->contactAutomations($contact);

        $contactAutomation = current(array_filter($contactAutomations, function ($contactAutomation) use ($automation) {
            return $contactAutomation->automation == $automation->id;
        }));

        if (empty($contactAutomation)) {
            return;
        }

        $this->delete("contactAutomations/{$contactAutomation->id}");

The method delete() does not exist on TestMonitor\ActiveCampaign\Actions\ManagesContacts. Did you maybe mean deleteContact()?

    }

    /**
     * Removing all automations from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     */
    public function removeAllAutomationsFromContact(Contact $contact)
    {
        $contactAutomations = $this->contactAutomations($contact);

        foreach ($contactAutomations as $contactAutomation) {
            $this->delete("contactAutomations/{$contactAutomation->id}");

The method delete() does not exist on TestMonitor\ActiveCampaign\Actions\ManagesContacts. Did you maybe mean deleteContact()?

        }
    }

    /**
     * Removing a tag from a contact.
     *
     * @param \TestMonitor\ActiveCampaign\Resources\Contact $contact
     * @param \TestMonitor\ActiveCampaign\Resources\Tag $tag
     */
    public function removeTagFromContact(Contact $contact, Tag $tag)

This method seems to be duplicated in your project.

    {
        $contactTags = $this->contactTags($contact);

        $contactTag = current(array_filter($contactTags, function ($contactTag) use ($tag) {
            return $contactTag->tag == $tag->id;
        }));

        if (empty($contactTag)) {
            return;
        }

        $this->delete("contactTags/{$contactTag->id}");

The method delete() does not exist on TestMonitor\ActiveCampaign\Actions\ManagesContacts. Did you maybe mean deleteContact()?

    }

    /**
     * Determines the contact ID.
     *
     * @param Contact|int|string $contact
     */
    protected function getContactId($contact)
    {
        if ($contact instanceof Contact) {
            return $contact->id;
        } elseif (is_numeric($contact)) {
            return (int) $contact;
        } else {
            $contact = $this->findContact($contact);

            return $contact->id;
        }
    }
}