SettingsController::__construct() - Code Metrics - Inspection of "Added admin control panel stuff" - gitaminhq/Gitamin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#81)

by Phecho

created 2015-12-12 03:56 UTC

SettingsController::__construct() B

↳ Parent: SettingsController

Complexity

Conditions	1
Paths	1

Size

Total Lines	46
Code Lines	35

Duplication

Lines	46
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	46
loc	46
rs	8.9412
cc	1
eloc	35
nc	1
nop	0

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

use Exception;
use Gitamin\Models\Setting;
use GrahamCampbell\Binput\Facades\Binput;
use Illuminate\Routing\Controller;
use Illuminate\Support\Facades\Lang;
use Illuminate\Support\Facades\Redirect;
use Illuminate\Support\Facades\Session;
use Illuminate\Support\Facades\View;

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

    /**
     * Creates a new settings controller instance.
     *
     * @return void

     */
    public function __construct()

    {
        $this->subMenu = [
            'overview' => [
                'title'  => 'Overview',
                'url'    => route('admin.index'),
                'icon'   => 'fa fa-wrench',
                'active' => false,
            ],
            'general' => [
                'title'  => trans('admin.settings.general.general'),
                'url'    => route('admin.settings.general'),
                'icon'   => 'fa fa-gear',
                'active' => false,
            ],
            'theme' => [
                'title'  => trans('admin.settings.theme.theme'),
                'url'    => route('admin.settings.theme'),
                'icon'   => 'fa fa-list-alt',
                'active' => false,
            ],
            'stylesheet' => [
                'title'  => trans('admin.settings.stylesheet.stylesheet'),
                'url'    => route('admin.settings.stylesheet'),
                'icon'   => 'fa fa-magic',
                'active' => false,
            ],
            'localization' => [
                'title'  => trans('admin.settings.localization.localization'),
                'url'    => route('admin.settings.localization'),
                'icon'   => 'fa fa-language',
                'active' => false,
            ],
            'timezone' => [
                'title'  => trans('admin.settings.timezone.timezone'),
                'url'    => route('admin.settings.timezone'),
                'icon'   => 'fa fa-calendar',
                'active' => false,
            ],
        ];

        View::share([
            'sub_title' => trans('admin.settings.settings'),
            'sub_menu'  => $this->subMenu,
        ]);
    }

    /**
     * Shows the settings general view.
     *
     * @return \Illuminate\View\View
     */
    public function showGeneralView()

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

        Session::flash('redirect_to', $this->subMenu['general']['url']);

        return View::make('admin.settings.general')
            ->withPageTitle(trans('admin.settings.general.general').' - '.trans('admin.admin'))
            ->withSubMenu($this->subMenu);
    }

    /**
     * Shows the settings localization view.
     *
     * @return \Illuminate\View\View
     */
    public function showLocalizationView()

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

        Session::flash('redirect_to', $this->subMenu['localization']['url']);

        return View::make('admin.settings.localization')
            ->withPageTitle(trans('admin.settings.localization.localization').' - '.trans('admin.admin'))
            ->withSubMenu($this->subMenu);
    }

    /**
     * Shows the settings theme view.
     *
     * @return \Illuminate\View\View
     */
    public function showThemeView()

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

        Session::flash('redirect_to', $this->subMenu['theme']['url']);

        return View::make('admin.settings.theme')
            ->withPageTitle(trans('admin.settings.theme.theme').' - '.trans('admin.admin'))
            ->withSubMenu($this->subMenu);
    }

    /**
     * Shows the settings timezone view.
     *
     * @return \Illuminate\View\View
     */
    public function showTimezoneView()

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

        Session::flash('redirect_to', $this->subMenu['timezone']['url']);

        return View::make('admin.settings.timezone')
            ->withPageTitle(trans('admin.settings.timezone.timezone').' - '.trans('admin.admin'))
            ->withSubMenu($this->subMenu);
    }

    /**
     * Shows the settings stylesheet view.
     *
     * @return \Illuminate\View\View
     */
    public function showStylesheetView()

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

        Session::flash('redirect_to', $this->subMenu['stylesheet']['url']);

        return View::make('admin.settings.stylesheet')
            ->withPageTitle(trans('admin.settings.stylesheet.stylesheet').' - '.trans('admin.admin'))
            ->withSubMenu($this->subMenu);
    }

    /**
     * Updates the system settings.
     *
     * @return \Illuminate\View\View
     */
    public function postSettings()
    {
        $redirectUrl = Session::get('redirect_to', route('admin.settings.general'));

        if (Binput::get('remove_banner') === '1') {
            $setting = Setting::where('name', 'app_banner');
            $setting->delete();
        }

        if (Binput::hasFile('app_banner')) {
            $file = Binput::file('app_banner');

            // Image Validation.
            // Image size in bytes.
            $maxSize = $file->getMaxFilesize();

            if ($file->getSize() > $maxSize) {
                return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.general.too-big', ['size' => $maxSize]));
            }

            if (!$file->isValid() || $file->getError()) {
                return Redirect::to($redirectUrl)->withErrors($file->getErrorMessage());
            }

            if (!starts_with($file->getMimeType(), 'image/')) {
                return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.general.images-only'));
            }

            // Store the banner.
            Setting::firstOrCreate(['name' => 'app_banner'])->update(['value' => base64_encode(file_get_contents($file->getRealPath()))]);

            // Store the banner type
            Setting::firstOrCreate(['name' => 'app_banner_type'])->update(['value' => $file->getMimeType()]);
        }

        try {
            foreach (Binput::except(['app_banner', 'remove_banner']) as $settingName => $settingValue) {
                Setting::firstOrCreate(['name' => $settingName])->update(['value' => $settingValue]);
            }
        } catch (Exception $e) {
            return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.edit.failure'));
        }

        if (Binput::has('app_locale')) {
            Lang::setLocale(Binput::get('app_locale'));
        }

        return Redirect::to($redirectUrl)->withSuccess(trans('admin.settings.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\Admin;
13
14		use Exception;
15		use Gitamin\Models\Setting;
16		use GrahamCampbell\Binput\Facades\Binput;
17		use Illuminate\Routing\Controller;
18		use Illuminate\Support\Facades\Lang;
19		use Illuminate\Support\Facades\Redirect;
20		use Illuminate\Support\Facades\Session;
21		use Illuminate\Support\Facades\View;
22
23		class SettingsController extends Controller
24		{
25		/**
26		* Array of sub-menu items.
27		*
28		* @var array
29		*/
30		protected $subMenu = [];
31
32		/**
33		* Creates a new settings controller instance.
34		*
35		* @return void
		0 ignored issues – show Comprehensibility Best Practice introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
36		*/
37	View Code Duplication	public function __construct()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
38		{
39		$this->subMenu = [
40		'overview' => [
41		'title' => 'Overview',
42		'url' => route('admin.index'),
43		'icon' => 'fa fa-wrench',
44		'active' => false,
45		],
46		'general' => [
47		'title' => trans('admin.settings.general.general'),
48		'url' => route('admin.settings.general'),
49		'icon' => 'fa fa-gear',
50		'active' => false,
51		],
52		'theme' => [
53		'title' => trans('admin.settings.theme.theme'),
54		'url' => route('admin.settings.theme'),
55		'icon' => 'fa fa-list-alt',
56		'active' => false,
57		],
58		'stylesheet' => [
59		'title' => trans('admin.settings.stylesheet.stylesheet'),
60		'url' => route('admin.settings.stylesheet'),
61		'icon' => 'fa fa-magic',
62		'active' => false,
63		],
64		'localization' => [
65		'title' => trans('admin.settings.localization.localization'),
66		'url' => route('admin.settings.localization'),
67		'icon' => 'fa fa-language',
68		'active' => false,
69		],
70		'timezone' => [
71		'title' => trans('admin.settings.timezone.timezone'),
72		'url' => route('admin.settings.timezone'),
73		'icon' => 'fa fa-calendar',
74		'active' => false,
75		],
76		];
77
78		View::share([
79		'sub_title' => trans('admin.settings.settings'),
80		'sub_menu' => $this->subMenu,
81		]);
82		}
83
84		/**
85		* Shows the settings general view.
86		*
87		* @return \Illuminate\View\View
88		*/
89	View Code Duplication	public function showGeneralView()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
90		{
91		$this->subMenu['general']['active'] = true;
92
93		Session::flash('redirect_to', $this->subMenu['general']['url']);
94
95		return View::make('admin.settings.general')
96		->withPageTitle(trans('admin.settings.general.general').' - '.trans('admin.admin'))
97		->withSubMenu($this->subMenu);
98		}
99
100		/**
101		* Shows the settings localization view.
102		*
103		* @return \Illuminate\View\View
104		*/
105	View Code Duplication	public function showLocalizationView()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
106		{
107		$this->subMenu['localization']['active'] = true;
108
109		Session::flash('redirect_to', $this->subMenu['localization']['url']);
110
111		return View::make('admin.settings.localization')
112		->withPageTitle(trans('admin.settings.localization.localization').' - '.trans('admin.admin'))
113		->withSubMenu($this->subMenu);
114		}
115
116		/**
117		* Shows the settings theme view.
118		*
119		* @return \Illuminate\View\View
120		*/
121	View Code Duplication	public function showThemeView()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
122		{
123		$this->subMenu['theme']['active'] = true;
124
125		Session::flash('redirect_to', $this->subMenu['theme']['url']);
126
127		return View::make('admin.settings.theme')
128		->withPageTitle(trans('admin.settings.theme.theme').' - '.trans('admin.admin'))
129		->withSubMenu($this->subMenu);
130		}
131
132		/**
133		* Shows the settings timezone view.
134		*
135		* @return \Illuminate\View\View
136		*/
137	View Code Duplication	public function showTimezoneView()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
138		{
139		$this->subMenu['timezone']['active'] = true;
140
141		Session::flash('redirect_to', $this->subMenu['timezone']['url']);
142
143		return View::make('admin.settings.timezone')
144		->withPageTitle(trans('admin.settings.timezone.timezone').' - '.trans('admin.admin'))
145		->withSubMenu($this->subMenu);
146		}
147
148		/**
149		* Shows the settings stylesheet view.
150		*
151		* @return \Illuminate\View\View
152		*/
153	View Code Duplication	public function showStylesheetView()
		0 ignored issues – show Duplication introduced 2015-12-12 03:59 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
154		{
155		$this->subMenu['stylesheet']['active'] = true;
156
157		Session::flash('redirect_to', $this->subMenu['stylesheet']['url']);
158
159		return View::make('admin.settings.stylesheet')
160		->withPageTitle(trans('admin.settings.stylesheet.stylesheet').' - '.trans('admin.admin'))
161		->withSubMenu($this->subMenu);
162		}
163
164		/**
165		* Updates the system settings.
166		*
167		* @return \Illuminate\View\View
168		*/
169		public function postSettings()
170		{
171		$redirectUrl = Session::get('redirect_to', route('admin.settings.general'));
172
173		if (Binput::get('remove_banner') === '1') {
174		$setting = Setting::where('name', 'app_banner');
175		$setting->delete();
176		}
177
178		if (Binput::hasFile('app_banner')) {
179		$file = Binput::file('app_banner');
180
181		// Image Validation.
182		// Image size in bytes.
183		$maxSize = $file->getMaxFilesize();
184
185		if ($file->getSize() > $maxSize) {
186		return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.general.too-big', ['size' => $maxSize]));
187		}
188
189		if (!$file->isValid() \|\| $file->getError()) {
190		return Redirect::to($redirectUrl)->withErrors($file->getErrorMessage());
191		}
192
193		if (!starts_with($file->getMimeType(), 'image/')) {
194		return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.general.images-only'));
195		}
196
197		// Store the banner.
198		Setting::firstOrCreate(['name' => 'app_banner'])->update(['value' => base64_encode(file_get_contents($file->getRealPath()))]);
199
200		// Store the banner type
201		Setting::firstOrCreate(['name' => 'app_banner_type'])->update(['value' => $file->getMimeType()]);
202		}
203
204		try {
205		foreach (Binput::except(['app_banner', 'remove_banner']) as $settingName => $settingValue) {
206		Setting::firstOrCreate(['name' => $settingName])->update(['value' => $settingValue]);
207		}
208		} catch (Exception $e) {
209		return Redirect::to($redirectUrl)->withErrors(trans('admin.settings.edit.failure'));
210		}
211
212		if (Binput::has('app_locale')) {
213		Lang::setLocale(Binput::get('app_locale'));
214		}
215
216		return Redirect::to($redirectUrl)->withSuccess(trans('admin.settings.edit.success'));
217		}
218		}
219

gitaminhq / Gitamin

Pull Request — master (#81)

SettingsController::__construct() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like