Gitamin\Http\Controllers\Dashboard\MomentController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	54
Duplicated Lines	40.74 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	22	22	1
A	index()	0	10	1

<?php

/*
 * This file is part of Gitamin.
 * 
 * Copyright (C) 2015-2016 The Gitamin Team
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Gitamin\Http\Controllers\Dashboard;

use Gitamin\Models\Moment;
use Illuminate\Routing\Controller;
use Illuminate\Support\Facades\View;

class MomentController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */

    /**
     * Array of sub-menu items.
     *
     * @var array
     */
    protected $subMenu = [];

    /**
     * Creates a new moment controller 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()

This method seems to be duplicated in your project.

    {
        $this->subMenu = [
            'moments' => [
                'title'  => trans('dashboard.projects.yours'),
                'url'    => route('dashboard.moments.index'),
                'icon'   => 'fa fa-sliders',
                'active' => false,
            ],
            'project_update' => [
                'title'  => trans('dashboard.projects.starred'),
                'url'    => route('dashboard.moments.index'),
                'icon'   => 'fa fa-edit',
                'active' => false,
            ],
        ];

        View::share([
            'sub_menu'  => $this->subMenu,
            'sub_title' => trans_choice('dashboard.moments.moments', 2),
        ]);
    }

    public function index()
    {
        $moments = Moment::recent()->get();

The method recent() does not exist on Gitamin\Models\Moment. Did you maybe mean scopeRecent()?

        $this->subMenu['moments']['active'] = true;

        return View::make('dashboard.moments.index')
            ->withPageTitle('Moments')
            ->withMoments($moments)
            ->withSubMenu($this->subMenu);
    }
}