CommerceGuys\AuthNet\ARBCreateSubscriptionRequest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	setSubscriptionRequest()	0	5	1
A	attachData()	0	4	1

<?php

namespace CommerceGuys\AuthNet;

use GuzzleHttp\Client;
use CommerceGuys\AuthNet\DataTypes\SubscriptionRequest;
use CommerceGuys\AuthNet\Request\RequestInterface;

/**
 * Use this method to create subscriptions using Automated Recurring Billing.
 *
 * @link https://developer.authorize.net/api/reference/#recurring-billing
 */
class ARBCreateSubscriptionRequest extends BaseApiRequest
{
    protected $subscriptionRequest;

    public function __construct(
        Configuration $configuration,
        Client $client,
        Subscription $subscription = null
    ) {
        parent::__construct($configuration, $client);
        $this->subscriptionRequest = $subscription;
    }

    /**
     * @param \CommerceGuys\AuthNet\DataTypes\SubscriptionRequest $subscriptionRequest
     * @return $this
     */
    public function setSubscriptionRequest(SubscriptionRequest $subscriptionRequest)

The parameter $subscriptionRequest is not used and could be removed.

    {
        $this->subscriptionRequest = $subscription;

The variable $subscription does not exist. Did you mean $subscriptionRequest?

        return $this;
    }

    protected function attachData(RequestInterface $request)
    {
        $request->addDataType($this->subscriptionRequest);

It seems like $this->subscriptionRequest can also be of type null; however, CommerceGuys\AuthNet\Request\RequestInterface::addDataType() does only seem to accept object<CommerceGuys\AuthNet\DataTypes\DataTypeInterface>, maybe add an additional type check?

    }
}