TestMonitor\ActiveCampaign\Actions\ManagesAccounts

Complexity

Total Complexity

10

Size/Duplication

Total Lines	117
Duplicated Lines	23.93 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	accounts()	0	8	1
A	findAccount()	0	10	1
A	createAccount()	0	9	1
A	updateAccount()	0	9	1
A	findOrCreateAccount()	14	14	3
A	updateOrCreateAccount()	14	14	3

<?php

namespace TestMonitor\ActiveCampaign\Actions;

use TestMonitor\ActiveCampaign\Resources\Account;

trait ManagesAccounts
{
    use ImplementsActions;

    /**
     * Get all accounts.
     *
     * @return array
     */
    public function accounts()
    {
        return $this->transformCollection(
            $this->get('accounts'),
            Account::class,
            'accounts'
        );
    }

    /**
     * Find account by name.
     *
     * @param string $name
     *
     * @return Account|null
     */
    public function findAccount($name)
    {
        $accounts = $this->transformCollection(
            $this->get('accounts', ['query' => ['search' => $name]]),
            Account::class,
            'accounts'
        );

        return array_shift($accounts);
    }

    /**
     * Create new account.
     *
     * @param array $data
     *
     * @return Account|null
     */
    public function createAccount(array $data = [])
    {
        $accounts = $this->transformCollection(
            $this->post('accounts', ['json' => ['account' => $data]]),
            Account::class
        );

        return array_shift($accounts);
    }

    /**
     * Update an account.
     *
     * @param int $accountId
     * @param array $data
     *
     * @return Account|null
     */
    public function updateAccount($accountId, array $data = [])
    {
        $accounts = $this->transformCollection(
            $this->put("accounts/{$accountId}", ['json' => ['account' => $data]]),
            Account::class
        );

        return array_shift($accounts);
    }

    /**
     * Find or create an account.
     *
     * @param string $name
     * @param array $data
     *
     * @return Account
     */
    public function findOrCreateAccount($name, array $data = [])

This method seems to be duplicated in your project.

    {
        $account = $this->findAccount($name);

        if ($account instanceof Account) {
            return $account;
        }

        if (empty($data)) {
            $data = ['name' => $name];
        }

        return $this->createAccount($data);
    }

    /**
     * Update or create an account.
     *
     * @param string $name
     * @param array $data
     *
     * @return Account
     */
    public function updateOrCreateAccount($name, array $data = [])

This method seems to be duplicated in your project.

    {
        $account = $this->findAccount($name);

        if ($account instanceof Account) {
            return $this->updateAccount($account->id, $data);
        }

        if (empty($data)) {
            $data = ['name' => $name];
        }

        return $this->createAccount($data);
    }
}