AlterLtUsersTable::up() - Code Metrics - Inspection of "change static var to env" - xoco70/laravel-tournaments - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( d9783c...375e5d )

by Julien

created 2018-10-10 12:49 UTC

AlterLtUsersTable::up() B

↳ Parent: AlterLtUsersTable

Complexity

Conditions	6
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	22
rs	8.9457
c	0
b	0
f	0
cc	6
nc	1
nop	0

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Config;
use Illuminate\Support\Facades\Schema;

class AlterLtUsersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::table(config('user.table'), function (Blueprint $table) {
            if (!Schema::hasColumn('name')) {
                $table->string('name')->default('name');
            }
            if (!Schema::hasColumn('firstname')) {
                $table->string('firstname')->default('firstname');
            }
            if (!Schema::hasColumn('lastname')) {
                $table->string('lastname')->default('lastname');
            }
            if (!Schema::hasColumn('email')) {
                $table->string('email')->unique();
            }

            if (!Schema::hasColumn('password')) {
                $table->string('password', 60);
            }
        });

    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('users');
    }
}


1			<?php
2
3			use Illuminate\Database\Migrations\Migration;
4			use Illuminate\Database\Schema\Blueprint;
5			use Illuminate\Support\Facades\Config;
6			use Illuminate\Support\Facades\Schema;
7
8			class AlterLtUsersTable extends Migration
9			{
10			/**
11			* Run the migrations.
12			*
13			* @return void
14			*/
15			public function up()
16			{
17			Schema::table(config('user.table'), function (Blueprint $table) {
18			if (!Schema::hasColumn('name')) {
19			$table->string('name')->default('name');
20			}
21			if (!Schema::hasColumn('firstname')) {
22			$table->string('firstname')->default('firstname');
23			}
24			if (!Schema::hasColumn('lastname')) {
25			$table->string('lastname')->default('lastname');
26			}
27			if (!Schema::hasColumn('email')) {
28			$table->string('email')->unique();
29			}
30
31			if (!Schema::hasColumn('password')) {
32			$table->string('password', 60);
33			}
34			});
35
36			}
37
38			/**
39			* Reverse the migrations.
40			*
41			* @return void
42			*/
43			public function down()
44			{
45			Schema::dropIfExists('users');
46			}
47			}
48

xoco70 / laravel-tournaments

Push — master ( d9783c...375e5d )

AlterLtUsersTable::up() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like