CreateEventDateTimesTable::up() - Code Metrics - Inspection of "Added commands and actions" - dbrax/eticket - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — main ( f9c7f4...aea9b1 )

by Emmanuel

created 2021-03-31 17:06 UTC

CreateEventDateTimesTable::up() A

↳ Parent: CreateEventDateTimesTable

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	18
rs	9.6666
c	0
b	0
f	0
cc	1
nc	1
nop	0

_<?php


    /**
     * Author: Emmanuel Paul Mnzava
     * Twitter: @epmnzava
     * Github:https://github.com/dbrax/eticket
     * Email: [email protected]
     * 
     */

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

    class CreateEventDateTimesTable extends Migration
    {
        /**
         * Run the migrations.
         *
         * @return void
         */
        public function up()
        {
            Schema::create('event_datetimes', function (Blueprint $table) {
                $table->bigIncrements('id');
                $table->unsignedBigInteger('eventid')->nullable();
                $table->foreign('eventid')->references('id')->on('events');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

                $table->foreignId('eventid')
                    ->constrained()
                    ->onUpdate('cascade')
                    ->onDelete('cascade');
                $table->date('event_date');
                $table->time('event_time', $precision = 0);
                $table->date('event_end_date')->nullable();
                $table->time('event_end_time', $precision = 0)->nullable();
                $table->timestamps();
            });
        }

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


1			_<?php
2
3
4			/**
5			* Author: Emmanuel Paul Mnzava
6			* Twitter: @epmnzava
7			* Github:https://github.com/dbrax/eticket
8			* Email: [email protected]
9			*
10			*/
11
12			use Illuminate\Support\Facades\Schema;
13			use Illuminate\Database\Schema\Blueprint;
14			use Illuminate\Database\Migrations\Migration;
15
16			class CreateEventDateTimesTable extends Migration
17			{
18			/**
19			* Run the migrations.
20			*
21			* @return void
22			*/
23			public function up()
24			{
25			Schema::create('event_datetimes', function (Blueprint $table) {
26			$table->bigIncrements('id');
27			$table->unsignedBigInteger('eventid')->nullable();
28			$table->foreign('eventid')->references('id')->on('events');
			0 ignored issues – show Bug introduced 2021-03-31 17:12 UTC by Report Bug Copy Issue Report The method `references` does only exist in `Illuminate\Database\Schema\ForeignKeyDefinition`, but not in `Illuminate\Support\Fluent`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
29
30			$table->foreignId('eventid')
31			->constrained()
32			->onUpdate('cascade')
33			->onDelete('cascade');
34			$table->date('event_date');
35			$table->time('event_time', $precision = 0);
36			$table->date('event_end_date')->nullable();
37			$table->time('event_end_time', $precision = 0)->nullable();
38			$table->timestamps();
39			});
40			}
41
42			/**
43			* Reverse the migrations.
44			*
45			* @return void
46			*/
47			public function down()
48			{
49			Schema::dropIfExists('event_datetimes');
50			}
51			}
52

dbrax / eticket

Push — main ( f9c7f4...aea9b1 )

CreateEventDateTimesTable::up() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like