Hborras\TwitterAdsSDK\TwitterAds\Campaign\Feature

Complexity

Total Complexity

6

Size/Duplication

Total Lines	53
Duplicated Lines	32.08 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	assureApi()	10	10	3
A	all()	7	7	1
A	getTwitterAds()	0	4	1

<?php
/**
 * Created by PhpStorm.
 * User: hborras
 * Date: 11/01/18
 * Time: 20:06
 */

namespace Hborras\TwitterAdsSDK\TwitterAds\Campaign;


use Hborras\TwitterAdsSDK\TwitterAds;
use Hborras\TwitterAdsSDK\TwitterAds\Cursor;

class Feature
{

    const RESOURCE_COLLECTION = 'features';
    const RESOURCE_REPLACE    = '{account_id}';

    /** @var  TwitterAds $twitterAds */
    private $twitterAds;

    /**
     * Feature constructor.
     * @param TwitterAds $twitterAds
     */
    public function __construct(TwitterAds $twitterAds)
    {

        $this->twitterAds = static::assureApi($twitterAds);
    }

    protected static function assureApi(TwitterAds $instance = null)

This method seems to be duplicated in your project.

    {
        $instance = $instance ?: TwitterAds::instance();
        if (!$instance) {
            throw new \InvalidArgumentException(
                'An Api instance must be provided as argument or ' .
                'set as instance in the \TwitterAds\Api');
        }
        return $instance;
    }

    /**
     * Returns a Cursor instance for a given resource.
     *
     * @param $params
     *
     * @return Cursor
     */
    public function all($params = [])

This method seems to be duplicated in your project.

    {
        $resource = str_replace(static::RESOURCE_REPLACE, $this->getTwitterAds()->getAccountId(), static::RESOURCE_COLLECTION);
        $response = $this->getTwitterAds()->get($resource, $params);

        return new Cursor($response->getBody()->data, $this->getTwitterAds(), $response->getBody(), $params);
    }

    /**
     * @return TwitterAds
     */
    public function getTwitterAds()
    {
        return $this->twitterAds;
    }
}