WorkEnvironmentSeeder::run() - Code Metrics - Inspection of "Reverting messy merges of gb_postres into dev" - GCTC-NTGC/TalentCloud - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — dev (#297)

by Tristan

created 2018-08-27 14:50 UTC

WorkEnvironmentSeeder::run() A

↳ Parent: WorkEnvironmentSeeder

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	10
dl	0
loc	16
rs	9.9332
c	2
b	0
f	0
cc	2
nc	2
nop	0

<?php

use Illuminate\Database\Seeder;
use App\Models\Manager;
use App\Models\WorkEnvironment;
use App\Models\Lookup\Frequency;

class WorkEnvironmentSeeder extends Seeder
namespace YourVendor;

class YourClass { }
{
    protected $faker;

    public function __construct(Faker\Generator $faker) {
        $this->faker = $faker;
    }

    /**
     * Run the Users table seeds.
     *
     * @return void
     */
    public function run()
    {
        $faker = $this->faker;


        //Find all managers that don't have a work environment yet
        $managers = Manager::doesntHave('work_environment')->get();

        foreach($managers as $manager) {
            $workEnvironment = new WorkEnvironment();
            $workEnvironment->manager_id = $manager->id;
            $workEnvironment->fill([
                'remote_work_allowed' => $this->faker->boolean(),
                'telework_allowed_frequency_id' => Frequency::inRandomOrder()->first()->id,
                'flexible_hours_frequency_id' => Frequency::inRandomOrder()->first()->id,
            ]);
            $workEnvironment->save();
        }
    }
}


1			<?php
2
3			use Illuminate\Database\Seeder;
4			use App\Models\Manager;
5			use App\Models\WorkEnvironment;
6			use App\Models\Lookup\Frequency;
7
8			class WorkEnvironmentSeeder extends Seeder
			0 ignored issues – show Coding Style Compatibility introduced 2018-08-24 17:34 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
9			{
10			protected $faker;
11
12			public function __construct(Faker\Generator $faker) {
13			$this->faker = $faker;
14			}
15
16			/**
17			* Run the Users table seeds.
18			*
19			* @return void
20			*/
21			public function run()
22			{
23			$faker = $this->faker;
			0 ignored issues – show Unused Code introduced 2018-08-24 17:34 UTC by Report Bug Copy Issue Report The assignment to `$faker` is dead and can be removed. Loading history...
24
25			//Find all managers that don't have a work environment yet
26			$managers = Manager::doesntHave('work_environment')->get();
27
28			foreach($managers as $manager) {
29			$workEnvironment = new WorkEnvironment();
30			$workEnvironment->manager_id = $manager->id;
31			$workEnvironment->fill([
32			'remote_work_allowed' => $this->faker->boolean(),
33			'telework_allowed_frequency_id' => Frequency::inRandomOrder()->first()->id,
34			'flexible_hours_frequency_id' => Frequency::inRandomOrder()->first()->id,
35			]);
36			$workEnvironment->save();
37			}
38			}
39			}
40

GCTC-NTGC / TalentCloud

Pull Request — dev (#297)

WorkEnvironmentSeeder::run() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like