murrant / librenmsv2

app/Http/Requests/UpdateUserRequest.php

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         if ($this->input('password')) {
             $rules = ['password'              => 'required|min:5|max:255',
-                      'password_confirmation' => 'required|same:password',
+                        'password_confirmation' => 'required|same:password',
             ];
             if (!Auth::user()->isAdmin()) {
                 $rules['current_password'] = 'required|password:user_id';

app/Http/Controllers/UserController.php

Doc Comments +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     /**
      * Update a notifications status.
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function update($id, $action)
     {
@@ -55,7 +55,7 @@ 
     /**
      * Create a new notification
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function create(Request $request)
     {

app/Http/Controllers/UserPortController.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(AdminOnlyRequest $request, $user_id)
     {
@@ -69,7 +69,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function destroy(AdminOnlyRequest $request, $user_id, $port_id)
     {

app/Http/Controllers/UserDeviceController.php

Doc Comments +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(AdminOnlyRequest $request, $user_id)
     {
@@ -69,7 +69,7 @@ 
      * @param AdminOnlyRequest $request
      * @param $user_id
      * @param $port_id
-     * @return mixed
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function destroy(AdminOnlyRequest $request, $user_id, $port_id)
     {

app/Http/Controllers/Widgets/WidgetDataController.php

Braces +8 added lines, -4 removed lines

@@ -82,7 +82,8 @@ 
         $uptime = Settings::get('uptime_warning', 84600);
         if ($request->user()->hasGlobalRead()) {
             $devices = Device::where('ignore', '=', 0)->get();
-        } else {
+        }
+        else {
             $devices = User::find($request->user()->user_id)->devices()->where('ignore', '=', 0)->get();
         }
         $count = ['warn' => 0, 'up' => 0, 'down' => 0];
@@ -90,10 +91,12 @@ 
             if ($device->status == 1) {
                 if (($device->uptime < $uptime) && ($device->uptime != '0')) {
                     $count['warn']++;
-                } else {
+                }
+                else {
                     $count['up']++;
                 }
-            } else {
+            }
+            else {
                 $count['down']++;
             }
         }
@@ -122,7 +125,8 @@ 
             $count['ports']['down'] = Port::isDown()->count();
             $count['ports']['ignored'] = Port::isIgnored()->count();
             $count['ports']['disabled'] = Port::isDisabled()->count();
-        } else {
+        }
+        else {
             $user = User::find($request->user()->user_id);
 
             $count['devices']['total'] = $user->devices()->count();

app/Api/Controllers/DashboardWidgetController.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
      */
     public function show(Request $request, $id)
     {
-        $widget  = Widgets::find($id);
+        $widget = Widgets::find($id);
         return array('widget' => $widget);
     }
 

app/DataTables/DeviceDataTable.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace App\DataTables;
 
-use App\Models\DeviceGroup;
 use App\Models\Device;
 
 class DeviceDataTable extends BaseDataTable

app/Http/Controllers/DeviceController.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Http\Controllers;
 
 use App\DataTables\DeviceDataTable;
-use App\DataTables\DeviceGroupDataTable;
 use App\Models\DeviceGroup;
 use Dingo\Api\Http;
 use Dingo\Api\Routing\Helpers;

Spacing +2 added lines, -2 removed lines

@@ -18,10 +18,10 @@
      *
      * @return \Illuminate\Http\JsonResponse|\Illuminate\View\View
      */
-    public function index(DeviceDataTable $dataTable, $group_id=-1)
+    public function index(DeviceDataTable $dataTable, $group_id = -1)
     {
         $group_name = "";
-        if($group_id >= 0) {
+        if ($group_id >= 0) {
             $dataTable->addScope(new \App\DataTables\Scopes\DeviceGroup($group_id));
             $group_name = DeviceGroup::find($group_id)->name;
         }

app/DataTables/Scopes/DeviceGroup.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     /**
      * DeviceGroup constructor.
      *
-     * @param $group_id
+     * @param integer $group_id
      */
     function __construct($group_id)
     {