uidaho / smartsettia

app/Device.php

Doc Comments +1 added lines, -1 removed lines

@@ -330,7 +330,7 @@
      * Get the page number of the device for the dashboard device table pagination
      *
      * @param int $limit
-     * @return int
+     * @return double
      */
     public function dashPageNum($limit)
     {

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
      *
      * @var array
      */
-    protected $hidden = ['token'];
+    protected $hidden = [ 'token' ];
     
     /**
      * The attributes that are mass assignable.
@@ -50,7 +50,7 @@ 
      *
      * @var array
      */
-    protected static $ignoreChangedAttributes = ['updated_at'];
+    protected static $ignoreChangedAttributes = [ 'updated_at' ];
     
     /**
      * The attributes to log in the Activity Log
@@ -91,7 +91,7 @@ 
      */
     public function getSiteAttribute()
     {
-        return $this->location->site ?? (object)[];
+        return $this->location->site ?? (object) [ ];
     }
     
     /**
@@ -151,7 +151,7 @@ 
         else
             $time = new Carbon($value, 'UTC');
         
-        $this->attributes['open_time'] = $time->format('H:i:s');
+        $this->attributes[ 'open_time' ] = $time->format('H:i:s');
     }
     
     /**
@@ -173,7 +173,7 @@ 
         else
             $time = new Carbon($value, 'UTC');
         
-        $this->attributes['close_time'] = $time->format('H:i:s');
+        $this->attributes[ 'close_time' ] = $time->format('H:i:s');
     }
     
     /**
@@ -299,7 +299,7 @@ 
         $isOpen = $this->cover_status === 'open';
         $isClosed = $this->cover_status === 'closed';
             
-        switch ($this['cover_command'])
+        switch ($this[ 'cover_command' ])
         {
             case 'open':
                 if ($isOpen)

Braces +28 added lines, -22 removed lines

@@ -107,8 +107,9 @@ 
         $time = new Carbon($value, 'UTC');
         
         //If the user is logged in then use there preferred timezone
-        if (Auth::check())
-            $time = $time->setTimezone(Auth::user()->timezone);
+        if (Auth::check()) {
+                    $time = $time->setTimezone(Auth::user()->timezone);
+        }
 
         return $time->format('H:i');
     }
@@ -126,8 +127,9 @@ 
         $time = new Carbon($value, 'UTC');
     
         //If the user is logged in then use there preferred timezone
-        if (Auth::check())
-            $time = $time->setTimezone(Auth::user()->timezone);
+        if (Auth::check()) {
+                    $time = $time->setTimezone(Auth::user()->timezone);
+        }
         
         return $time->format('H:i');
     }
@@ -147,9 +149,9 @@ 
         {
             $time = new Carbon($value, Auth::user()->timezone);
             $time = $time->setTimezone('UTC');
+        } else {
+                    $time = new Carbon($value, 'UTC');
         }
-        else
-            $time = new Carbon($value, 'UTC');
         
         $this->attributes['open_time'] = $time->format('H:i:s');
     }
@@ -169,9 +171,9 @@ 
         {
             $time = new Carbon($value, Auth::user()->timezone);
             $time = $time->setTimezone('UTC');
+        } else {
+                    $time = new Carbon($value, 'UTC');
         }
-        else
-            $time = new Carbon($value, 'UTC');
         
         $this->attributes['close_time'] = $time->format('H:i:s');
     }
@@ -282,10 +284,11 @@ 
         $time_now = Carbon::now($timezone);
     
         //Check if the current time is during the open schedule or not
-        if ($time_now->gt($open_time) && $time_now->lt($close_time))
-            return true;
-        else
-            return false;
+        if ($time_now->gt($open_time) && $time_now->lt($close_time)) {
+                    return true;
+        } else {
+                    return false;
+        }
     }
     
     /**
@@ -302,16 +305,18 @@ 
         switch ($this['cover_command'])
         {
             case 'open':
-                if ($isOpen)
-                    $status = 'open';
-                else
-                    $status = 'opening';
+                if ($isOpen) {
+                                    $status = 'open';
+                } else {
+                                    $status = 'opening';
+                }
                 break;
             case 'close':
-                if ($isClosed)
-                    $status = 'closed';
-                else
-                    $status = 'closing';
+                if ($isClosed) {
+                                    $status = 'closed';
+                } else {
+                                    $status = 'closing';
+                }
                 break;
             case 'lock':
                 $status = 'locked';
@@ -320,8 +325,9 @@ 
                 $status = 'error';
         }
     
-        if ($this->cover_status === 'error')
-            $status = 'error';
+        if ($this->cover_status === 'error') {
+                    $status = 'error';
+        }
         
         return $status;
     }

app/Http/Controllers/DashboardController.php

Unused Use Statements -3 removed lines

@@ -2,13 +2,10 @@
 
 namespace App\Http\Controllers;
 
-use Illuminate\Support\Facades\Auth;
 use Illuminate\Http\Request;
-use Validator;
 use App\Device;
 use App\Site;
 use App\Location;
-use Illuminate\Support\HtmlString;
 
 class DashboardController extends Controller
 {

Spacing +6 added lines, -6 removed lines

@@ -46,7 +46,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;
@@ -66,13 +66,13 @@ 
             ->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()
-            ->where('location_id', '=', $locations[0]->id ?? 0)
+            ->where('location_id', '=', $locations[ 0 ]->id ?? 0)
             ->orderBy('name', 'ASC')
             ->paginate(4);
     
@@ -80,13 +80,13 @@ 
         $active_device = $devices->where('id', $device_id)->first();
         //Set the active device to the first device in $devices if it is not empty and the original active device wasn't found
         if (!$devices->isEmpty() && $active_device == null)
-            $active_device = $devices[0];
+            $active_device = $devices[ 0 ];
     
         //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 +12 added lines, -10 removed lines

@@ -45,13 +45,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;
@@ -79,8 +79,9 @@ 
         //Get the active device
         $active_device = $devices->where('id', $device_id)->first();
         //Set the active device to the first device in $devices if it is not empty and the original active device wasn't found
-        if (!$devices->isEmpty() && $active_device == null)
-            $active_device = $devices[0];
+        if (!$devices->isEmpty() && $active_device == null) {
+                    $active_device = $devices[0];
+        }
     
         //Store the active site, location, and device in a collection
         $active_data = collect(['device' => $active_device, 'location' => $locations[0] ?? null, 'site' => $sites[0] ?? null]);
@@ -88,12 +89,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 ]);
+        }
     }
     
     /**