seregazhuk\PinterestBot\Api\Traits\HasRelatedTopics

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getRelatedTopics()	0	7	1
A	getPinsFor()	0	9	1
	getFeedRequestData()	0	1	?
A	getFeedUrl()	0	4	2

<?php

namespace seregazhuk\PinterestBot\Api\Traits;

use Generator;
use seregazhuk\PinterestBot\Helpers\UrlBuilder;

/**
 * Trait HasRelatedTopics
 * @package seregazhuk\PinterestBot\Api\Traits
 *
 * @property string $feedUrl
 */
trait HasRelatedTopics
{
    use HandlesRequest;

    /**
     * Returns a list of related topics
     * @param string $interest
     * @return array|bool
     */
    public function getRelatedTopics($interest)
    {
        return $this->execGetRequest(
            ['interest_name' => $interest],
            UrlBuilder::RESOURCE_GET_CATEGORIES_RELATED
        );
    }

    /**
     * Returns a feed of pins.
     *
     * @param string $interest
     * @param int $limit
     * @return Generator
     */
    public function getPinsFor($interest, $limit = 0)
    {
        $params = [
            'data' => $this->getFeedRequestData($interest),
            'url'  => $this->getFeedUrl(),
        ];

        return $this->getPaginatedResponse($params, $limit);

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

    }

    /**
     * @param $interest
     * @return array
     */
    abstract protected function getFeedRequestData($interest);

    /**
     * @return string
     */
    protected function getFeedUrl()
    {
        return property_exists($this, 'feedUrl') ? $this->feedUrl : '';
    }
}