DepartmentsController::index() - Code Metrics - Inspection of "Merge pull request #238 from idevelopment/patch-23..." - idevelopment/timecontrol - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — development ( 76b4cb...55251d )

by Glenn

created 2016-05-14 20:53 UTC

DepartmentsController::index() A

↳ Parent: DepartmentsController

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	5
rs	9.4285
cc	1
eloc	3
nc	1
nop	0

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests;
use App\Departments;
use App\DepartmentMembers;
use Mail;
use App\User;
use App\Http\Controllers\Controller;

/**
 * Class DepartmentsController
 * @package App\Http\Controllers
 */
class DepartmentsController extends Controller
{
    /**
     * DepartmentsController Constructor.
     */
    public function __construct()
    {
        $this->middleware('auth');
        $this->middleware('lang');
    }

    /**
     * Get the departments index.
     *
     * @return view
     */ 
    public function index()
    {
        $departments = Departments::orderBy('department_name', 'asc')->with('managers')->paginate(10);
        return view('departments/list', ['departments' => $departments]);
    }

    /**
     * Create a new department.
     * 
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function create()
    {
        $managers = User::all();
        return view('departments/create', ['managers' => $managers]);
    }

    /**
     * Store a newly created resource in storage.
     *
     * @param  Requests\DepartmentsValidator|Request $request
     * @return \Illuminate\Http\Response
     */
    public function store(Requests\DepartmentsValidator $request)
    {
        // TODO: Check for mass assign - SQL queries.
        
        $departments = new Departments;
        $departments->department_name = $request->get('department_name');
        $departments->department_manager = $request->get('department_manager');
        $departments->department_description = $request->get('department_description');
        $departments->save();

        $department_id = $departments->id;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

        $manager = new DepartmentMembers;
        $manager->departmentid = $department_id;
        $manager->userid = $request->get('department_manager');
        $manager->save();

        session()->flash('message', trans('FlashSession.departmentNew'));
        return redirect('staff/departments');
    }

    /**
     * Display the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function show($id)

    {
        //
    }

    /**
     * Show the form for editing the specified resource.
     *
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function edit($id)
    {
        $data['query'] = Departments::all($id);
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        return view('departments/edit', $data);
    }

    /**
     * Update the specified resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  int  $id
     * @return \Illuminate\Http\Response
     */
    public function update(Request $request, $id)

    {
        Departments::all($id)->update(/* Values */);

        return redirect()->back(302);
    }

    /**
     * Search a department in the database.
     *
     * @param Request $request
     * @return \Illuminate\Http\Response
     */
    public function search(Request $request)
    {
        $term = $request->get('name');

        if (empty($term)) {
            return redirect('staff/departments', 302);
        }

        $data['departments'] = Departments::where('department_name', 'LIKE', "%$term%")
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
            ->orderBy('department_name', 'asc')
            ->with('managers')
            ->paginate(10);

        return view('departments/list', $data);
    }

    /**
     * Remove the specified resource from storage.
     *
     * @param  Request $request
     * @return \Illuminate\Http\Response
     */
    public function destroy(Request $request)
    {
        $query = Departments::destroy($request->get('department_id'));
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        session()->flash('message', trans('FlashSession.departmentDestroy'));

        return redirect()->back(302);
    }
}


1			<?php
2
3			namespace App\Http\Controllers;
4
5			use Illuminate\Http\Request;
6			use App\Http\Requests;
7			use App\Departments;
8			use App\DepartmentMembers;
9			use Mail;
10			use App\User;
11			use App\Http\Controllers\Controller;
12
13			/**
14			* Class DepartmentsController
15			* @package App\Http\Controllers
16			*/
17			class DepartmentsController extends Controller
18			{
19			/**
20			* DepartmentsController Constructor.
21			*/
22			public function __construct()
23			{
24			$this->middleware('auth');
25			$this->middleware('lang');
26			}
27
28			/**
29			* Get the departments index.
30			*
31			* @return view
32			*/
33			public function index()
34			{
35			$departments = Departments::orderBy('department_name', 'asc')->with('managers')->paginate(10);
36			return view('departments/list', ['departments' => $departments]);
37			}
38
39			/**
40			* Create a new department.
41			*
42			* @return \Illuminate\Contracts\View\Factory\|\Illuminate\View\View
43			*/
44			public function create()
45			{
46			$managers = User::all();
47			return view('departments/create', ['managers' => $managers]);
48			}
49
50			/**
51			* Store a newly created resource in storage.
52			*
53			* @param Requests\DepartmentsValidator\|Request $request
54			* @return \Illuminate\Http\Response
55			*/
56			public function store(Requests\DepartmentsValidator $request)
57			{
58			// TODO: Check for mass assign - SQL queries.
59
60			$departments = new Departments;
61			$departments->department_name = $request->get('department_name');
62			$departments->department_manager = $request->get('department_manager');
63			$departments->department_description = $request->get('department_description');
64			$departments->save();
65
66			$department_id = $departments->id;
			0 ignored issues – show Documentation introduced 2016-03-03 00:42 UTC by Report Bug Copy Issue Report The property `id` does not exist on `object<App\Departments>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
67
68			$manager = new DepartmentMembers;
69			$manager->departmentid = $department_id;
70			$manager->userid = $request->get('department_manager');
71			$manager->save();
72
73			session()->flash('message', trans('FlashSession.departmentNew'));
74			return redirect('staff/departments');
75			}
76
77			/**
78			* Display the specified resource.
79			*
80			* @param int $id
81			* @return \Illuminate\Http\Response
82			*/
83			public function show($id)
			0 ignored issues – show Unused Code introduced 2016-03-03 00:42 UTC by Report Bug Copy Issue Report The parameter `$id` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
84			{
85			//
86			}
87
88			/**
89			* Show the form for editing the specified resource.
90			*
91			* @param int $id
92			* @return \Illuminate\Http\Response
93			*/
94			public function edit($id)
95			{
96			$data['query'] = Departments::all($id);
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-05-14 15:28 UTC by Report Bug Copy Issue Report `$data` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$data = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
97			return view('departments/edit', $data);
98			}
99
100			/**
101			* Update the specified resource in storage.
102			*
103			* @param \Illuminate\Http\Request $request
104			* @param int $id
105			* @return \Illuminate\Http\Response
106			*/
107			public function update(Request $request, $id)
			0 ignored issues – show Unused Code introduced 2016-03-03 00:42 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
108			{
109			Departments::all($id)->update(/* Values */);
110
111			return redirect()->back(302);
112			}
113
114			/**
115			* Search a department in the database.
116			*
117			* @param Request $request
118			* @return \Illuminate\Http\Response
119			*/
120			public function search(Request $request)
121			{
122			$term = $request->get('name');
123
124			if (empty($term)) {
125			return redirect('staff/departments', 302);
126			}
127
128			$data['departments'] = Departments::where('department_name', 'LIKE', "%$term%")
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-05-03 10:05 UTC by Report Bug Copy Issue Report `$data` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$data = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
129			->orderBy('department_name', 'asc')
130			->with('managers')
131			->paginate(10);
132
133			return view('departments/list', $data);
134			}
135
136			/**
137			* Remove the specified resource from storage.
138			*
139			* @param Request $request
140			* @return \Illuminate\Http\Response
141			*/
142			public function destroy(Request $request)
143			{
144			$query = Departments::destroy($request->get('department_id'));
			0 ignored issues – show Unused Code introduced 2016-05-05 11:27 UTC by Report Bug Copy Issue Report `$query` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
145			session()->flash('message', trans('FlashSession.departmentDestroy'));
146
147			return redirect()->back(302);
148			}
149			}
150

idevelopment / timecontrol

Push — development ( 76b4cb...55251d )

DepartmentsController::index() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like