librenms / librenmsv2

app/Providers/RouteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function map(Router $router)
     {
-        $router->group(['namespace' => $this->namespace], function ($router) {
+        $router->group(['namespace' => $this->namespace], function($router) {
             require app_path('Http/routes.php');
         });
     }

app/Api/Controllers/DeviceController.php

Braces +2 added lines, -4 removed lines

@@ -19,8 +19,7 @@ 
     public function list_devices(Request $request) {
         if ($request->user()->level >= 10 || $request->user()->level == 5) {
             return Device::all();
-        }
-        else {
+        } else {
             return User::find($request->user()->user_id)->devices()->get();
         }
     }
@@ -31,8 +30,7 @@ 
     public function list_ports(Request $request) {
         if ($request->user()->level >= 10 || $request->user()->level == 5) {
             return Port::all();
-        }
-        else {
+        } else {
             return User::find($request->user()->user_id)->ports()->get();
         }
     }

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         }
         else {
             $user = User::find($request->user()->user_id);
-            $device =  $user->devices()->find($id);
+            $device = $user->devices()->find($id);
         }
         // morph the data as required
         if ($request->query('displayFormat') == 'link') {

app/User.php

Doc Comments +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
     /**
      * Test if the User is an admin or demo.
      *
-     * @return \Illuminate\Database\Eloquent\Builder
+     * @return boolean
      */
     public function isAdmin()
     {
@@ -42,7 +42,7 @@ 
      * Test if this user has global read access
      * these users have a level of 5, 10 or 11 (demo).
      *
-     * @return \Illuminate\Database\Eloquent\Builder
+     * @return boolean
      */
     public function hasGlobalRead()
     {

app/Api/Controllers/PortController.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use App\User;
 use App\Port;
 use App\Api\Transformers\PortTransformer;
-
 use Illuminate\Http\Request;
 use Dingo\Api\Routing\Helpers;
 use Illuminate\Support\Facades\Input;

Braces +3 added lines, -6 removed lines

@@ -30,14 +30,12 @@ 
             $includes = explode(',', Input::get('include'));
             if (in_array('device', $includes)) {
                 $ports = Port::with('device')->get();
-            }
-            else {
+            } else {
                 $ports = Port::all();
             }
             return $this->response->collection($ports, new PortTransformer);
             return $ports;
-        }
-        else {
+        } else {
             return Auth::user()->ports()->get();
         }
     }
@@ -73,8 +71,7 @@ 
     {
         if ($request->user()->hasGlobalRead()) {
             return Port::find($id);
-        }
-        else {
+        } else {
             $user = User::find($request->user()->user_id);
             return $user->ports()->find($id);
         }

app/Http/Controllers/NotificationController.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
     public function list_devices(Request $request) {
         if ($request->user()->level >= 10 || $request->user()->level == 5) {
             return Devices::all();
-        }
-        else {
+        } else {
             return User::find($request->user()->user_id)->devices()->get();
         }
     }

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Illuminate\Contracts\Auth\Guard;
 use Dingo\Api\Http;
 use Dingo\Api\Routing\Helpers;
-use Illuminate\Http\Request;
 
 class SetViewVariable
 {

app/Api/Controllers/NotificationController.php

Braces +4 added lines, -8 removed lines

@@ -27,8 +27,7 @@ 
         if ($type === 'archive')
         {
             $notifications = Notification::read([$request->user()->user_id])->limit()->get();
-        }
-        else {
+        } else {
             $notifications = Notification::unread()->limit()->get();
         }
 
@@ -51,19 +50,16 @@ 
                 if ($read->save())
                 {
                     return $this->response->array(array('statusText'=>'OK'));
-                }
-                else {
+                } else {
                     return $this->response->errorInternal();
                 }
             }
-        }
-        elseif ($action === 'unread' || $action === 'unsticky')
+        } elseif ($action === 'unread' || $action === 'unsticky')
         {
             if (NotificationAttrib::where('notifications_id', $id)->delete() >= 0)
             {
                 return $this->response->array(array('statusText'=>'OK'));
-            }
-            else {
+            } else {
                 return $this->response->errorInternal();
             }
         }

app/Notification.php

Spacing +2 added lines, -2 removed lines

@@ -41,11 +41,11 @@
 
     public function scopeLimit($query)
     {
-        return $query->select('notifications.*','key');
+        return $query->select('notifications.*', 'key');
     }
 
     public function users() {
-        return $this->belongsToMany('App\User')->withPivot('notifications_id','user_id');
+        return $this->belongsToMany('App\User')->withPivot('notifications_id', 'user_id');
     }
 
 }