seregazhuk\Favro\Api\Api

Complexity

Total Complexity

10

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	7
Bugs	1	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__get()	0	10	2
A	setOrganization()	0	8	2
A	setOrganizationId()	0	6	1
A	getOrganizationId()	0	4	1
A	getOrganizationByName()	0	12	3

<?php

namespace seregazhuk\Favro\Api;

use seregazhuk\Favro\Api\Endpoints\Cards;
use seregazhuk\Favro\Api\Endpoints\Tasks;
use seregazhuk\Favro\Api\Endpoints\Users;
use seregazhuk\Favro\Api\Endpoints\Columns;
use seregazhuk\Favro\Api\Endpoints\Widgets;
use seregazhuk\Favro\Api\Endpoints\Endpoint;
use seregazhuk\Favro\Api\Endpoints\Collections;
use seregazhuk\Favro\Api\Endpoints\Organizations;
use seregazhuk\Favro\Api\Endpoints\EndpointsContainer;

/**
 * Class Api
 *
 * @property Organizations $organizations
 * @property Users $users
 * @property Collections $collections
 * @property Widgets $widgets
 * @property Columns $columns
 * @property Cards $cards
 * @property Tasks $tasks
 */
class Api
{
    /**
     * @var EndpointsContainer
     */
    protected $endpointsContainer;

    /**
     * @var string
     */
    protected $organizationId;

    public function __construct(EndpointsContainer $endpointsContainer)
    {
        $this->endpointsContainer = $endpointsContainer;
    }

    /**
     * Magic method to access different endpoints.
     *
     * @param string $endpoint
     *
     * @return Endpoint
     */
    public function __get($endpoint)
    {
        $endpoint = $this->endpointsContainer->resolve($endpoint);

        if (method_exists($endpoint, 'setOrganizationId')) {
            $endpoint->setOrganizationId($this->organizationId);
        }

        return $endpoint;
    }

    /**
     * @param $organizationName
     * @return $this
     */
    public function setOrganization($organizationName)
    {
        if($organization = $this->getOrganizationByName($organizationName)) {
            $this->setOrganizationId($organization['organizationId']);
        }

        return $this;
    }

    /**
     * @param int $organizationId
     * @return $this
     */
    public function setOrganizationId($organizationId)
    {
        $this->organizationId = $organizationId;

The property $organizationId was declared of type string, but $organizationId is of type integer. Maybe add a type cast?

        return $this;
    }


    /**
     * @return string
     */
    public function getOrganizationId()
    {
        return $this->organizationId;
    }

    /**
     * @param $organization
     * @return bool|array
     */
    protected function getOrganizationByName($organization)
    {
        $organizations = $this->organizations->getAll();

        foreach ($organizations['entities'] as $entity) {
            if ($entity['name'] == $organization) {
                return $organization;
            }
        }

        return false;
    }
}