App\Console\Commands\GeneratePageRoutes

Complexity

Total Complexity

3

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	handle()	0	12	2

<?php

namespace App\Console\Commands;

use App\Classes\PageRouteBuilder;
use App\Classes\Repositories\MenuRepository;
use App\Classes\Repositories\PageRepository;
use App\Model\Menu;
use App\Model\Page;
use Illuminate\Console\Command;

class GeneratePageRoutes extends Command
{

    /**
     * @var MenuRepository
     */
    private $menus;

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'page.routes.generate';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generate the page routes based on the linked menus.';

    /**
     * Create a new command instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(MenuRepository $menus)
    {
        parent::__construct();

        $this->menus = $menus;
    }

    /**
     * Execute the console command.
     *
     * @return mixed

Consider making the return type a bit more specific; maybe use boolean.

     */
    public function handle()
    {
        /** @var Menu $menu */
        foreach ($this->menus->all() as $menu)
        {
            PageRouteBuilder::link($menu->page, $menu);

            $this->info("{$menu->title} was linked to {$menu->page->slug}");
        }

        return true;
    }
}