uidaho / smartsettia

app/Http/Controllers/ApiController.php

Unused Use Statements -2 removed lines

@@ -9,8 +9,6 @@
 use App\User;
 use App\Sensor;
 use App\SensorData;
-use Illuminate\Support\Facades\Cache;
-use Illuminate\Support\Facades\Storage;
 use Carbon\Carbon;
 
 class ApiController extends Controller

app/Http/Controllers/DashboardController.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Http\Request;
-use Validator;
 use App\Device;
 use App\Site;
 use App\Location;

Spacing +10 added lines, -10 removed lines

@@ -44,7 +44,7 @@ 
             
             //If the location exists then set the pagination page to be where the device is located
             if ($location != null)
-                $request->merge(['page' => $device->dashPageNum(4)]);
+                $request->merge([ 'page' => $device->dashPageNum(4) ]);
             $site_id = $location->site_id ?? 0;
             $location_id = $location->id ?? 0;
             $device_id = $device->id ?? 0;
@@ -64,14 +64,14 @@ 
             ->get();
         //Get all locations for the selected site with the selected location first
         $locations = Location::select('id', 'name', 'site_id')
-            ->where('site_id', '=', $sites[0]->id ?? 0)
+            ->where('site_id', '=', $sites[ 0 ]->id ?? 0)
             ->orderByRaw("id = ? DESC", $location_id)
             ->orderBy('name', 'ASC')
             ->get();
         //Get all devices for the selected location
         $devices = Device::publicDashData()
             ->leftJoin('deviceimages as image', 'devices.id', '=', 'image.device_id')
-            ->where('location_id', '=', $locations[0]->id ?? 0)
+            ->where('location_id', '=', $locations[ 0 ]->id ?? 0)
             ->orderBy('name', 'ASC')
             ->paginate(4);
     
@@ -84,33 +84,33 @@ 
             //If the original device with the given device id was not found
             //Then assign the the first device in the device list as the active device
             if ($active_device == null)
-                $active_device = $devices[0];
+                $active_device = $devices[ 0 ];
         
             //Add an attribute to each device defining if it is stale
-            $devices->transform(function ($item)
+            $devices->transform(function($item)
             {
                 //Mark the device as stale if the device has missed three updates plus a minute
                 $deviceStaleMins = ceil(($item->update_rate * 3) / 60) + 1;
-                $item['isDeviceStale'] = ($item->last_network_update_at <= Carbon::now()->subMinute($deviceStaleMins)) ? true : false;
+                $item[ 'isDeviceStale' ] = ($item->last_network_update_at <= Carbon::now()->subMinute($deviceStaleMins)) ? true : false;
     
                 //Mark the device image as stale if the device has missed three image updates plus a minute
                 if ($item->image_updated_at != null)
                 {
                     $imageStaleMins = ceil(($item->image_rate * 3) / 60) + 1;
-                    $item['isImageStale'] = ($item->image_updated_at <= Carbon::now()->subMinute($imageStaleMins)) ? true : false;
+                    $item[ 'isImageStale' ] = ($item->image_updated_at <= Carbon::now()->subMinute($imageStaleMins)) ? true : false;
                 }
                 else
-                    $item['isImageStale'] = false;
+                    $item[ 'isImageStale' ] = false;
                 
                 return $item;
             });
         }
     
         //Store the active site, location, and device in a collection
-        $active_data = collect(['device' => $active_device, 'location' => $locations[0] ?? null, 'site' => $sites[0] ?? null]);
+        $active_data = collect([ 'device' => $active_device, 'location' => $locations[ 0 ] ?? null, 'site' => $sites[ 0 ] ?? null ]);
     
         //Use the device_list.blade.php to generate the device table html
-        $html_device_table = view('dashboard.device_list', ['devices' => $devices, 'active_data' => $active_data])->render();
+        $html_device_table = view('dashboard.device_list', [ 'devices' => $devices, 'active_data' => $active_data ])->render();
         
         if (\Request::ajax())
             return response()->json([ 'active_data' => $active_data, 'devices' => $devices, 'locations' => $locations,

Braces +14 added lines, -12 removed lines

@@ -43,13 +43,13 @@ 
             $location = $device->location ?? null;
             
             //If the location exists then set the pagination page to be where the device is located
-            if ($location != null)
-                $request->merge(['page' => $device->dashPageNum(4)]);
+            if ($location != null) {
+                            $request->merge(['page' => $device->dashPageNum(4)]);
+            }
             $site_id = $location->site_id ?? 0;
             $location_id = $location->id ?? 0;
             $device_id = $device->id ?? 0;
-        }
-        else
+        } else
         {
             //Get the active site, location, and device ids
             $site_id = $request->site_id ?? 0;
@@ -83,8 +83,9 @@ 
         {
             //If the original device with the given device id was not found
             //Then assign the the first device in the device list as the active device
-            if ($active_device == null)
-                $active_device = $devices[0];
+            if ($active_device == null) {
+                            $active_device = $devices[0];
+            }
         
             //Add an attribute to each device defining if it is stale
             $devices->transform(function ($item)
@@ -98,9 +99,9 @@ 
                 {
                     $imageStaleMins = ceil(($item->image_rate * 3) / 60) + 1;
                     $item['isImageStale'] = ($item->image_updated_at <= Carbon::now()->subMinute($imageStaleMins)) ? true : false;
+                } else {
+                                    $item['isImageStale'] = false;
                 }
-                else
-                    $item['isImageStale'] = false;
                 
                 return $item;
             });
@@ -112,12 +113,13 @@ 
         //Use the device_list.blade.php to generate the device table html
         $html_device_table = view('dashboard.device_list', ['devices' => $devices, 'active_data' => $active_data])->render();
         
-        if (\Request::ajax())
-            return response()->json([ 'active_data' => $active_data, 'devices' => $devices, 'locations' => $locations,
+        if (\Request::ajax()) {
+                    return response()->json([ 'active_data' => $active_data, 'devices' => $devices, 'locations' => $locations,
                 'sites' => $sites, 'html_device_table' => $html_device_table ]);
-        else
-            return view('dashboard.index', [ 'active_data' => $active_data, 'devices' => $devices,
+        } else {
+                    return view('dashboard.index', [ 'active_data' => $active_data, 'devices' => $devices,
                 'locations' => $locations, 'sites' => $sites ]);
+        }
     }
     
     /**