JobApis\Jobs\Client\Providers\MuseProvider

Complexity

Total Complexity

15

Size/Duplication

Total Lines	140
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
B	createJobObject()	0	24	1
A	getDefaultResponseFields()	0	18	1
A	getListingsPath()	0	4	1
A	setCategories()	0	11	3
A	setCompany()	0	7	3
A	setLevels()	0	11	3
A	setLocation()	0	7	3

<?php namespace JobApis\Jobs\Client\Providers;

use JobApis\Jobs\Client\Job;

class MuseProvider extends AbstractProvider
{
    /**
     * Returns the standardized job object
     *
     * @param array $payload Raw job payload from the API
     *
     * @return \JobApis\Jobs\Client\Job
     */
    public function createJobObject($payload = [])
    {
        $job = new Job([
            'title' => $payload['name'],
            'name' => $payload['name'],
            'description' => $payload['contents'],
            'url' => $payload['refs']['landing_page'],
            'sourceId' => $payload['id'],
        ]);

        // categories array
        $this->setCategories($job, $payload['categories']);

        // company array
        $this->setCompany($job, $payload['company']);

        // levels array
        $this->setLevels($job, $payload['levels']);

        // locations array
        $this->setLocation($job, $payload['locations']);

        return $job;
    }

    /**
     * Job response object default keys that should be set
     *
     * @return  string
     */
    public function getDefaultResponseFields()
    {
        return [

The return type of return array('levels', 'locations', 'tags', 'categories', 'publication_date', 'short_name', 'refs', 'contents', 'type', 'model_type', 'company', 'id', 'name'); (string[]) is incompatible with the return type declared by the abstract method JobApis\Jobs\Client\Providers\AbstractProvider::getDefaultResponseFields of type string.

            'levels', // array
            'locations', // array
            'tags', // array
            'categories', // array
            'publication_date',
            'short_name',
            'refs', // array
            'contents',
            'type',
            'model_type',
            'company', // array
            'id',
            'name',
        ];
    }

    /**
     * Get listings path
     *
     * @return  string
     */
    public function getListingsPath()
    {
        return 'results';
    }

    /**
     * Sets the categories on the job using the categories array
     *
     * @param Job $job
     * @param array $categories
     *
     * @return MuseProvider
     */
    protected function setCategories(Job $job, $categories = [])
    {
        $occupationalCats = [];
        foreach ($categories as $category) {
            $occupationalCats[] = $category['name'];
        }
        if ($occupationalCats) {

The expression $occupationalCats of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $job->setOccupationalCategory(implode(', ', $occupationalCats));
        }
        return $this;
    }

    /**
     * Sets the company on the job using the company array
     *
     * @param Job $job
     * @param array $company
     *
     * @return MuseProvider
     */
    protected function setCompany(Job $job, $company = [])
    {
        if ($company && isset($company['name'])) {

The expression $company of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $job->setCompany($company['name']);
        }
        return $this;
    }

    /**
     * Sets the experience levels on the job using the levels array
     *
     * @param Job $job
     * @param array $levels
     *
     * @return MuseProvider
     */
    protected function setLevels(Job $job, $levels = [])
    {
        $requirements = [];
        foreach ($levels as $level) {
            $requirements[] = $level['name'];
        }
        if ($requirements) {

The expression $requirements of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $job->setExperienceRequirements(implode(', ', $requirements));
        }
        return $this;
    }

    /**
     * Sets the location on the job using the first location in the array
     *
     * @param Job $job
     * @param array $locations
     *
     * @return MuseProvider
     */
    protected function setLocation(Job $job, $locations = [])
    {
        if (isset($locations[0]) && isset($locations[0]['name'])) {
            $job->setLocation($locations[0]['name']);
        }
        return $this;
    }
}