seregazhuk\HeadHunterApi\Traits\EmployerManagers

Complexity

Total Complexity

8

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getManagerTypes()	0	6	1
A	getManagers()	0	6	1
A	getManager()	0	6	1
A	getManagersWhoHasVacancies()	0	10	2
A	hasVacancies()	0	4	1
A	resolveEmployerId()	0	7	2

<?php

namespace seregazhuk\HeadHunterApi\Traits;

use UnexpectedValueException;

trait EmployerManagers
{
    use ResolvesCurrentUser;

    /**
     * Reference types and the rights of the Manager
     * @param string|bool $employerId
     * @return array
     */
    public function getManagerTypes($employerId = false)
    {
        $employerId = $this->resolveEmployerId($employerId);

        return $this->getResource("$employerId/manager_types", []);

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

    }

    /**
     * Get employer managers
     *
     * @param string|bool $employerId default resolved from profile
     * @return array
     */
    public function getManagers($employerId = false)
    {
        $employerId = $this->resolveEmployerId($employerId);

        return $this->getResource("$employerId/managers", []);

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

    }

    /**
     * Get manager information
     *
     * @param string $managerId
     * @param string|bool $employerId default resolved from profile
     * @return array
     */
    public function getManager($managerId, $employerId = false)
    {
        $employerId = $this->resolveEmployerId($employerId);

        return $this->getResource("$employerId/managers/$managerId", []);

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

    }

    /**
     * @param string|bool $employerId default resolved from profile
     * @return array
     */
    public function getManagersWhoHasVacancies($employerId = false)
    {
        $managers = $this->getManagers($employerId);

        if(!isset($managers['items']) ) {
            throw new UnexpectedValueException('Failed to get employer managers');
        }

        return array_filter($managers['items'], [$this, 'hasVacancies']);
    }

	/**
	 * @param array $manager
	 * @return bool
	 */
    protected function hasVacancies(array $manager)
    {
        return $manager['vacancies_count'] > 0;
    }

	/**
	 * @param string|bool $employerId
	 * @return string
	 */
    protected function resolveEmployerId($employerId)
    {
        return (false === $employerId)
            ? $this->getCurrentEmployerId()
            : $employerId
        ;
    }
}