uidaho / smartsettia

app/Http/Controllers/SensorController.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use App\Sensor;
 use App\DataTables\SensorDataTable;
 use Illuminate\Http\Request;
-use Validator;
 use Charts;
 
 class SensorController extends Controller 

app/Sensor.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      */
     public function getLatestDataAttribute()
     {
-        return $this->hasOne('App\SensorData')->latest()->first() ?? (object)[];
+        return $this->hasOne('App\SensorData')->latest()->first() ?? (object) [ ];
     }
     
     /**

app/Http/Controllers/DeviceController.php

Spacing +10 added lines, -10 removed lines

@@ -55,9 +55,9 @@ 
     {
         $device = Device::findOrFail($id);
         
-        $charts = [];
+        $charts = [ ];
         foreach ($device->sensors as $sensor) {
-            $charts[$sensor->id] = Charts::create('line', 'highcharts')
+            $charts[ $sensor->id ] = Charts::create('line', 'highcharts')
                 ->title($sensor->name)
                 ->elementLabel($sensor->type)
                 ->labels($sensor->last_week_daily_avg_data->pluck('date'))
@@ -89,7 +89,7 @@ 
         $sites = Site::select('id', 'name')->orderByRaw("id = ? DESC", $site_id)
             ->orderBy('name', 'ASC')->get();
         //Get all locations for the selected site with the selected location first
-        $locations = Location::select('id', 'name')->where('site_id', '=', $sites[0]->id ?? 0)
+        $locations = Location::select('id', 'name')->where('site_id', '=', $sites[ 0 ]->id ?? 0)
             ->orderByRaw("id = ? DESC", $location_id)->orderBy('name', 'ASC')->get();
         
         return view('device.edit', [ 'device' => $device, 'locations' => $locations, 'sites' => $sites ]);
@@ -110,7 +110,7 @@ 
         if (!empty($request->input('new_site_name')))
         {
             //Create a new site
-            $site = Site::create(['name' => $request->input('new_site_name')]);
+            $site = Site::create([ 'name' => $request->input('new_site_name') ]);
             $site_id = $site->id;
         }
         else
@@ -120,7 +120,7 @@ 
         if (!empty($request->input('new_location_name')))
         {
             //Create a new location
-            $location = Location::create(['name' => $request->input('new_location_name'), 'site_id' => $site_id]);
+            $location = Location::create([ 'name' => $request->input('new_location_name'), 'site_id' => $site_id ]);
             $location_id = $location->id;
         }
         else
@@ -147,9 +147,9 @@ 
             if ($request->command === 'unlock')
             {
                 if ($device->isDuringScheduleOpen())
-                    $command =  'open';
+                    $command = 'open';
                 else
-                    $command =  'close';
+                    $command = 'close';
             }
             $device->cover_command = $command;
         }
@@ -157,7 +157,7 @@ 
         $device->save();
     
         if (\Request::ajax())
-            return response()->json(['success' => 'Device updated successfully']);
+            return response()->json([ 'success' => 'Device updated successfully' ]);
         else
             return redirect()->route('device.show', $id)
                 ->with('success', 'Device updated successfully');
@@ -189,7 +189,7 @@ 
         }
 
         return redirect()->route('device.index')
-            ->with('success','Device deleted successfully');
+            ->with('success', 'Device deleted successfully');
     }
     
     /**
@@ -205,6 +205,6 @@ 
         $device->restore();
         
         return redirect()->route('device.show', $device->id)
-            ->with('success','Device restored successfully');
+            ->with('success', 'Device restored successfully');
     }
 }