Srmklive\Chargify\Traits\ChargifyAPI\Adjustments

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	adjust_subscription_balance()	0	14	1

<?php

namespace Srmklive\Chargify\Traits\ChargifyAPI;

trait Adjustments
{
    /**
     * Create an adjustment for a subscription.
     *
     * @param int    $subscription_id
     * @param int    $amount
     * @param string $memo
     *
     * @return array
     */
    public function adjust_subscription_balance($subscription_id, $amount, $memo): array
    {
        $this->options['json'] = [

The property options does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            'adjustment' => [
                'amount' => $amount,
                'memo'   => $memo,
            ]
        ];

        $this->apiEndPoint = "/subscriptions/{$subscription_id}/adjustments.json";

        $this->verb = 'post';

        return $this->doChargifyRequest();

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

    }
}