NavigationController::__construct() - Code Metrics - yajra/cms-core - Measure and Improve Code Quality continuously with Scrutinizer

NavigationController::__construct() A
last analyzed 2017-02-08 14:31 UTC

↳ Parent: NavigationController

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	0
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

namespace Yajra\CMS\Http\Controllers;

use Yajra\CMS\DataTables\NavigationDataTable;
use Yajra\CMS\Entities\Navigation;
use App\Http\Requests;
use Illuminate\Http\Request;

class NavigationController extends Controller
{
    /**
     * Controller specific permission ability map.
     *
     * @var array
     */
    protected $customPermissionMap = [
        'publish' => 'update',
    ];

    /**
     * NavigationController constructor.
     */
    public function __construct()
    {
        $this->authorizePermissionResource('navigation');
    }

    /**
     * Display list of navigation.
     *
     * @param \Yajra\CMS\DataTables\NavigationDataTable $dataTable
     * @return \Illuminate\Http\JsonResponse|\Illuminate\View\View
     */
    public function index(NavigationDataTable $dataTable)
    {
        return $dataTable->render('administrator.navigation.index');
    }

    /**
     * Show navigation form.
     *
     * @param \Yajra\CMS\Entities\Navigation $navigation
     * @return mixed
     */
    public function create(Navigation $navigation)
    {
        return view('administrator.navigation.create', compact('navigation'));
    }

    /**
     * Store a newly created navigation.
     *
     * @param \Illuminate\Http\Request $request
     * @return mixed
     */
    public function store(Request $request)

    {
        $this->validate($request, [
            'title' => 'required|max:255',
            'type'  => 'required|max:255|alpha|unique:navigation,type',
        ]);

        $navigation = new Navigation;
        $navigation->fill($request->all());
        $navigation->published = $request->get('published', false);
        $navigation->save();
        flash()->success(trans('cms::navigation.store.success'));


        return redirect()->route('administrator.navigation.index');
    }

    /**
     * Show and edit selected navigation.
     *
     * @param \Yajra\CMS\Entities\Navigation $navigation
     * @return mixed
     */
    public function edit(Navigation $navigation)
    {
        return view('administrator.navigation.edit', compact('navigation'));
    }

    /**
     * Update selected navigation.
     *
     * @param \Yajra\CMS\Entities\Navigation $navigation
     * @param \Illuminate\Http\Request $request
     * @return mixed
     */
    public function update(Navigation $navigation, Request $request)

    {
        $this->validate($request, [
            'title' => 'required|max:255',
            'type'  => 'required|max:255|alpha|unique:navigation,type,' . $navigation->id,
        ]);

        $navigation->fill($request->all());
        $navigation->published = $request->get('published', false);
        $navigation->save();
        flash()->success(trans('cms::navigation.update.success'));


        return redirect()->route('administrator.navigation.index');
    }

    /**
     * Remove selected navigation.
     *
     * @param \Yajra\CMS\Entities\Navigation $navigation
     * @return string
     * @throws \Exception
     */
    public function destroy(Navigation $navigation)
    {
        $navigation->delete();

        return $this->notifySuccess(trans('cms::navigation.destroy.success'));
    }

    /**
     * Publish/Unpublish a navigation.
     *
     * @param \Yajra\CMS\Entities\Navigation $navigation
     * @return \Illuminate\Http\JsonResponse
     */
    public function publish(Navigation $navigation)
    {
        $navigation->togglePublishedState();

        return $this->notifySuccess(sprintf(
            trans('cms::navigation.update.publish'),
            $navigation->published ? 'published' : 'unpublished'
        ));
    }
}


1		<?php
2
3		namespace Yajra\CMS\Http\Controllers;
4
5		use Yajra\CMS\DataTables\NavigationDataTable;
6		use Yajra\CMS\Entities\Navigation;
7		use App\Http\Requests;
8		use Illuminate\Http\Request;
9
10		class NavigationController extends Controller
11		{
12		/**
13		* Controller specific permission ability map.
14		*
15		* @var array
16		*/
17		protected $customPermissionMap = [
18		'publish' => 'update',
19		];
20
21		/**
22		* NavigationController constructor.
23		*/
24		public function __construct()
25		{
26		$this->authorizePermissionResource('navigation');
27		}
28
29		/**
30		* Display list of navigation.
31		*
32		* @param \Yajra\CMS\DataTables\NavigationDataTable $dataTable
33		* @return \Illuminate\Http\JsonResponse\|\Illuminate\View\View
34		*/
35		public function index(NavigationDataTable $dataTable)
36		{
37		return $dataTable->render('administrator.navigation.index');
38		}
39
40		/**
41		* Show navigation form.
42		*
43		* @param \Yajra\CMS\Entities\Navigation $navigation
44		* @return mixed
45		*/
46		public function create(Navigation $navigation)
47		{
48		return view('administrator.navigation.create', compact('navigation'));
49		}
50
51		/**
52		* Store a newly created navigation.
53		*
54		* @param \Illuminate\Http\Request $request
55		* @return mixed
56		*/
57	View Code Duplication	public function store(Request $request)
		0 ignored issues – show Duplication introduced 2016-06-13 15:10 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...
58		{
59		$this->validate($request, [
60		'title' => 'required\|max:255',
61		'type' => 'required\|max:255\|alpha\|unique:navigation,type',
62		]);
63
64		$navigation = new Navigation;
65		$navigation->fill($request->all());
66		$navigation->published = $request->get('published', false);
67		$navigation->save();
68		flash()->success(trans('cms::navigation.store.success'));
		0 ignored issues – show Bug introduced 2017-02-08 14:31 UTC by Report Bug Copy Issue Report It seems like `trans('cms::navigation.store.success')` targeting `trans()` can also be of type `object<Illuminate\Contra...Translation\Translator>`; however, `Laracasts\Flash\FlashNotifier::success()` does only seem to accept `string`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
69
70		return redirect()->route('administrator.navigation.index');
71		}
72
73		/**
74		* Show and edit selected navigation.
75		*
76		* @param \Yajra\CMS\Entities\Navigation $navigation
77		* @return mixed
78		*/
79		public function edit(Navigation $navigation)
80		{
81		return view('administrator.navigation.edit', compact('navigation'));
82		}
83
84		/**
85		* Update selected navigation.
86		*
87		* @param \Yajra\CMS\Entities\Navigation $navigation
88		* @param \Illuminate\Http\Request $request
89		* @return mixed
90		*/
91	View Code Duplication	public function update(Navigation $navigation, Request $request)
		0 ignored issues – show Duplication introduced 2016-06-13 15:10 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...
92		{
93		$this->validate($request, [
94		'title' => 'required\|max:255',
95		'type' => 'required\|max:255\|alpha\|unique:navigation,type,' . $navigation->id,
96		]);
97
98		$navigation->fill($request->all());
99		$navigation->published = $request->get('published', false);
100		$navigation->save();
101		flash()->success(trans('cms::navigation.update.success'));
		0 ignored issues – show Bug introduced 2017-02-08 14:31 UTC by Report Bug Copy Issue Report It seems like `trans('cms::navigation.update.success')` targeting `trans()` can also be of type `object<Illuminate\Contra...Translation\Translator>`; however, `Laracasts\Flash\FlashNotifier::success()` does only seem to accept `string`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
102
103		return redirect()->route('administrator.navigation.index');
104		}
105
106		/**
107		* Remove selected navigation.
108		*
109		* @param \Yajra\CMS\Entities\Navigation $navigation
110		* @return string
111		* @throws \Exception
112		*/
113		public function destroy(Navigation $navigation)
114		{
115		$navigation->delete();
116
117		return $this->notifySuccess(trans('cms::navigation.destroy.success'));
118		}
119
120		/**
121		* Publish/Unpublish a navigation.
122		*
123		* @param \Yajra\CMS\Entities\Navigation $navigation
124		* @return \Illuminate\Http\JsonResponse
125		*/
126		public function publish(Navigation $navigation)
127		{
128		$navigation->togglePublishedState();
129
130		return $this->notifySuccess(sprintf(
131		trans('cms::navigation.update.publish'),
132		$navigation->published ? 'published' : 'unpublished'
133		));
134		}
135		}
136

yajra / cms-core

NavigationController::__construct() A last analyzed 2017-02-08 14:31 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

NavigationController::__construct() A
last analyzed 2017-02-08 14:31 UTC