Signifly\Shopify\REST\Actions\ManagesShopifyPayments

Complexity

Total Complexity

5

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getDisputes()	0	3	1
A	getPayouts()	0	3	1
A	getDispute()	0	3	1
A	getPayout()	0	3	1
A	getBalance()	0	5	1

<?php

namespace Signifly\Shopify\REST\Actions;

use Illuminate\Support\Collection;
use Signifly\Shopify\REST\Resources\BalanceResource;
use Signifly\Shopify\REST\Resources\DisputeResource;
use Signifly\Shopify\REST\Resources\PayoutResource;
use Signifly\Shopify\Shopify;

/** @mixin Shopify */
trait ManagesShopifyPayments
{
    public function getBalance(): BalanceResource
    {
        $response = $this->get('shopify_payments/balance');

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

        return new BalanceResource($response['balance'], $this);
    }

    public function getDisputes(array $params = []): Collection
    {
        return $this->getResources('disputes', $params, ['shopify_payments']);

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

    }

    public function getDispute($disputeId): DisputeResource
    {
        return $this->getResource('disputes', $disputeId, ['shopify_payments']);

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

    }

    public function getPayouts(array $params = []): Collection
    {
        return $this->getResources('payouts', $params, ['shopify_payments']);
    }

    public function getPayout($payoutId): PayoutResource
    {
        return $this->getResource('payouts', $payoutId, ['shopify_payments']);
    }

}