CreateFirewallTable::up() - Code Metrics - Inspection of "Try to enable tests and coverage" - antonioribeiro/firewall - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (ecb760)

by Antonio Carlos

created 2017-09-04 05:39 UTC

CreateFirewallTable::up() A

↳ Parent: CreateFirewallTable

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	7
cts	7
cp	1
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	6
nc	1
nop	0
crap	1

<?php

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

class CreateFirewallTable extends Migration
namespace YourVendor;

class YourClass { }
{
    /**
     * Run the migration.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('firewall', function (Blueprint $table) {
            $table->increments('id');

            $table->string('ip_address', 39)->unique()->index();

            $table->boolean('whitelisted')->default(false); /// default is blacklist

            $table->timestamps();
        });
    }

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


1		<?php
2
3		use Illuminate\Database\Migrations\Migration;
4		use Illuminate\Database\Schema\Blueprint;
5		use Illuminate\Support\Facades\Schema;
6
7		class CreateFirewallTable extends Migration
		0 ignored issues – show Coding Style Compatibility introduced 2017-09-04 05:44 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 migration.
11		*
12		* @return void
13		*/
14		public function up()
15		{
16	38	Schema::create('firewall', function (Blueprint $table) {
17	38	$table->increments('id');
18
19	38	$table->string('ip_address', 39)->unique()->index();
20
21	38	$table->boolean('whitelisted')->default(false); /// default is blacklist
22
23	38	$table->timestamps();
24	38	});
25	38	}
26
27		/**
28		* Reverse the migration.
29		*
30		* @return void
31		*/
32	37	public function down()
33		{
34	37	Schema::dropIfExists('firewall');
35	37	}
36		}
37

antonioribeiro / firewall

Branch — master (ecb760)

CreateFirewallTable::up() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like