CreateActivationsTable::up() - Code Metrics - Inspection of "4ff80c60fd878196854ce7f9b16cb22f6ba9094a" - NoBrainerQuiz/web - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (4ff80c)

by Matt

created 2018-02-19 21:35 UTC

CreateActivationsTable::up() A

↳ Parent: CreateActivationsTable

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	12
nc	2
nop	0
dl	0
loc	16
rs	9.4285
c	1
b	0
f	0

<?php

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

class CreateActivationsTable extends Migration
namespace YourVendor;

class YourClass { }
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        if (Schema::hasTable('activations')) return; 

        Schema::create('activations', function (Blueprint $table) {
            $table->increments('id');
            $table->integer('user_id')->unsigned()->index();
            $table->string('token');
            $table->ipAddress('ip_address');
            $table->timestamps();

            $table->foreign('user_id')
                  ->references('id')
                  ->on('users')
                  ->onDelete('cascade')
                  ->onUpdate('cascade');
        });
    }

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


1			<?php
2
3			use Illuminate\Support\Facades\Schema;
4			use Illuminate\Database\Schema\Blueprint;
5			use Illuminate\Database\Migrations\Migration;
6
7			class CreateActivationsTable extends Migration
			0 ignored issues – show Coding Style Compatibility introduced 2018-02-19 19:00 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...
8			{
9			/**
10			* Run the migrations.
11			*
12			* @return void
13			*/
14			public function up()
15			{
16			if (Schema::hasTable('activations')) return;
17
18			Schema::create('activations', function (Blueprint $table) {
19			$table->increments('id');
20			$table->integer('user_id')->unsigned()->index();
21			$table->string('token');
22			$table->ipAddress('ip_address');
23			$table->timestamps();
24
25			$table->foreign('user_id')
26			->references('id')
27			->on('users')
28			->onDelete('cascade')
29			->onUpdate('cascade');
30			});
31			}
32
33			/**
34			* Reverse the migrations.
35			*
36			* @return void
37			*/
38			public function down()
39			{
40			Schema::dropIfExists('activations');
41			}
42			}
43

NoBrainerQuiz / web

Branch — master (4ff80c)

CreateActivationsTable::up() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like