RecruiterFilter::filter() - Code Metrics - jobapis/jobs-to-mail - Measure and Improve Code Quality continuously with Scrutinizer

RecruiterFilter::filter() A
last analyzed 2018-09-10 22:11 UTC

↳ Parent: RecruiterFilter

Complexity

Conditions	5
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5

Importance

Changes

Metric	Value
dl	0
loc	17
rs	9.3888
c	0
b	0
f	0
ccs	8
cts	8
cp	1
cc	5
nc	1
nop	2
crap	5

<?php namespace JobApis\JobsToMail\Filters;

use JobApis\Jobs\Client\Job;
use JobApis\JobsToMail\Models\Recruiter;
use JobApis\JobsToMail\Models\Search;

class RecruiterFilter
{
    public function __construct(Recruiter $recruiter)
    {
        $this->recruiter = $recruiter;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * Filters out jobs from recruiting companies if the user prefers it.
     *
     * @param array $jobs
     * @param Search $search
     *
     * @return array
     */
    public function filter(array $jobs, Search $search)
    {
        return array_filter($jobs, function (Job $job) use ($search) {
            // Make sure this job has a company
            if (isset($job->company) && $job->company) {
                // See if this company is not a recruiter
                if ($this->recruiter->whereNameLike($job->company)->first()) {
                    if ($search->no_recruiters === true) {
                        return false;
                    } else {
                        $job->setIndustry("Staffing");
                    }
                }
            }
            return true;
        });
    }
}


1		<?php namespace JobApis\JobsToMail\Filters;
2
3		use JobApis\Jobs\Client\Job;
4		use JobApis\JobsToMail\Models\Recruiter;
5		use JobApis\JobsToMail\Models\Search;
6
7		class RecruiterFilter
8		{
9	3	public function __construct(Recruiter $recruiter)
10		{
11	3	$this->recruiter = $recruiter;
		0 ignored issues – show Bug introduced 2016-10-28 20:45 UTC by Report Bug Copy Issue Report The property `recruiter` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
12	3	}
13
14		/**
15		* Filters out jobs from recruiting companies if the user prefers it.
16		*
17		* @param array $jobs
18		* @param Search $search
19		*
20		* @return array
21		*/
22		public function filter(array $jobs, Search $search)
23		{
24	3	return array_filter($jobs, function (Job $job) use ($search) {
25		// Make sure this job has a company
26	3	if (isset($job->company) && $job->company) {
27		// See if this company is not a recruiter
28	2	if ($this->recruiter->whereNameLike($job->company)->first()) {
29	2	if ($search->no_recruiters === true) {
30	1	return false;
31		} else {
32	1	$job->setIndustry("Staffing");
33		}
34		}
35		}
36	2	return true;
37	3	});
38		}
39		}
40

jobapis / jobs-to-mail

RecruiterFilter::filter() A last analyzed 2018-09-10 22:11 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

RecruiterFilter::filter() A
last analyzed 2018-09-10 22:11 UTC