ProfilesController::__construct() - Code Metrics - Inspection of "Moments related enhancements" - gitaminhq/Gitamin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#106)

by Phecho

created 2015-12-17 03:49 UTC

ProfilesController::__construct() A

↳ Parent: ProfilesController

Complexity

Conditions	1
Paths	1

Size

Total Lines	64
Code Lines	50

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	2

Metric	Value
c	3
b	0
f	2
dl	0
loc	64
rs	9.3957
cc	1
eloc	50
nc	1
nop	0

How to fix Long Method

<?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;

use Illuminate\Support\Facades\Redirect;
use Illuminate\Support\Facades\View;

class ProfilesController extends Controller
{
    /**
     * Array of sub-menu items.
     *
     * @var array
     */
    protected $subMenu = [];

    /**
     * Creates a new project controller instance.
     */
    public function __construct()
    {
        $this->subMenu = [
            'profiles' => [
                'title' => trans('gitamin.profiles.profiles'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-user',
                'active' => false,
            ],
            'account' => [
                'title' => trans('gitamin.profiles.account'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-gear',
                'active' => false,
            ],
            'emails' => [
                'title' => trans('gitamin.profiles.emails'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-envelope-o',
                'active' => false,
            ],
            'password' => [
                'title' => trans('gitamin.profiles.password'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-lock',
                'active' => false,
            ],
            'notifications' => [
                'title' => trans('gitamin.profiles.notifications'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-inbox',
                'active' => false,
            ],
            'ssh_keys' => [
                'title' => trans('gitamin.profiles.ssh_keys'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-key',
                'active' => false,
            ],
            'applications' => [
                'title' => trans('gitamin.profiles.applications'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-cloud',
                'active' => false,
            ],
            'preferences' => [
                'title' => trans('gitamin.profiles.preferences'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-image',
                'active' => false,
            ],
            'audit_log' => [
                'title' => trans('gitamin.profiles.audit_log'),
                'url' => route('profile.index'),
                'icon' => 'fa fa-history',
                'active' => false,
            ],
        ];

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

    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function indexAction()
    {
        //
        $this->subMenu['profiles']['active'] = true;

        return View::make('profiles.index')
            ->withSubMenu($this->subMenu)
            ->withPageTitle(trans('gitamin.profiles.profiles').' - '.trans('dashboard.dashboard'));
    }

    /**
     * Update the specified resource in storage.
     *
     * @return \Illuminate\Http\Response
     */
    public function updateAction()
    {
        // Do something

        return Redirect::route('profile.index')
            ->withSuccess(sprintf('%s %s', trans('dashboard.notifications.awesome'), trans('gitamin.profiles.edit.success')));
    }
}


1			<?php
2
3			/*
4			* This file is part of Gitamin.
5			*
6			* Copyright (C) 2015-2016 The Gitamin Team
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Gitamin\Http\Controllers;
13
14			use Illuminate\Support\Facades\Redirect;
15			use Illuminate\Support\Facades\View;
16
17			class ProfilesController extends Controller
18			{
19			/**
20			* Array of sub-menu items.
21			*
22			* @var array
23			*/
24			protected $subMenu = [];
25
26			/**
27			* Creates a new project controller instance.
28			*/
29			public function __construct()
30			{
31			$this->subMenu = [
32			'profiles' => [
33			'title' => trans('gitamin.profiles.profiles'),
34			'url' => route('profile.index'),
35			'icon' => 'fa fa-user',
36			'active' => false,
37			],
38			'account' => [
39			'title' => trans('gitamin.profiles.account'),
40			'url' => route('profile.index'),
41			'icon' => 'fa fa-gear',
42			'active' => false,
43			],
44			'emails' => [
45			'title' => trans('gitamin.profiles.emails'),
46			'url' => route('profile.index'),
47			'icon' => 'fa fa-envelope-o',
48			'active' => false,
49			],
50			'password' => [
51			'title' => trans('gitamin.profiles.password'),
52			'url' => route('profile.index'),
53			'icon' => 'fa fa-lock',
54			'active' => false,
55			],
56			'notifications' => [
57			'title' => trans('gitamin.profiles.notifications'),
58			'url' => route('profile.index'),
59			'icon' => 'fa fa-inbox',
60			'active' => false,
61			],
62			'ssh_keys' => [
63			'title' => trans('gitamin.profiles.ssh_keys'),
64			'url' => route('profile.index'),
65			'icon' => 'fa fa-key',
66			'active' => false,
67			],
68			'applications' => [
69			'title' => trans('gitamin.profiles.applications'),
70			'url' => route('profile.index'),
71			'icon' => 'fa fa-cloud',
72			'active' => false,
73			],
74			'preferences' => [
75			'title' => trans('gitamin.profiles.preferences'),
76			'url' => route('profile.index'),
77			'icon' => 'fa fa-image',
78			'active' => false,
79			],
80			'audit_log' => [
81			'title' => trans('gitamin.profiles.audit_log'),
82			'url' => route('profile.index'),
83			'icon' => 'fa fa-history',
84			'active' => false,
85			],
86			];
87
88			View::share([
89			'sub_menu' => $this->subMenu,
90			'sub_title' => trans_choice('dashboard.projects.projects', 2),
91			]);
92			}
93
94			/**
95			* Display a listing of the resource.
96			*
97			* @return \Illuminate\Http\Response
98			*/
99			public function indexAction()
100			{
101			//
102			$this->subMenu['profiles']['active'] = true;
103
104			return View::make('profiles.index')
105			->withSubMenu($this->subMenu)
106			->withPageTitle(trans('gitamin.profiles.profiles').' - '.trans('dashboard.dashboard'));
107			}
108
109			/**
110			* Update the specified resource in storage.
111			*
112			* @return \Illuminate\Http\Response
113			*/
114			public function updateAction()
115			{
116			// Do something
117
118			return Redirect::route('profile.index')
119			->withSuccess(sprintf('%s %s', trans('dashboard.notifications.awesome'), trans('gitamin.profiles.edit.success')));
120			}
121			}
122

gitaminhq / Gitamin

Pull Request — master (#106)

ProfilesController::__construct() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like