CreateLinksTable

Complexity

Total Complexity

2

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	up()	0	34	1
A	down()	0	3	1

<?php

use App\Model\Link;
use App\Model\Menu;
use App\Model\Page;
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use App\Classes\Repositories\MenuRepository;
use App\Classes\Repositories\PageRepository;
use Illuminate\Database\Migrations\Migration;

class CreateLinksTable extends Migration

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('links', function (Blueprint $table) {
            $table->increments('id');

            $table->unsignedInteger('from_id');
            $table->string('from_type');
            $table->index(['from_id', 'from_type']);

            $table->unsignedInteger('to_id')->nullable();
            $table->string('to_type')->nullable();
            $table->index(['to_id', 'to_type']);

            $table->string('external')->nullable();

            $table->timestamps();
        });

        Schema::table('menus', function (Blueprint $table) {
            $table->removeColumn('hyperlink');

            $table->removeColumn('page_id');
        });

        /**
         * Link the homepage menu to the homepage page.
         */
        $page = app(PageRepository::class)->whereID(1);
        $menu = app(MenuRepository::class)->whereID(1);

        /** @var Link $link */
        $link = app(Link::class);

        $link->connect($menu, $page)->save();

$page of type App\Model\Menu is incompatible with the type null|App\Classes\Interfaces\Linkable expected by parameter $object of App\Model\Link::connect().

    }

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