Completed
Branch travis-speedup (fadc4d)
by Tony
02:29
created
app/Api/Transformers/PortTransformer.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
     /**
36 36
      * Include Device
37 37
      *
38
-     * @return League\Fractal\ItemResource
38
+     * @return Fractal\Resource\Item
39 39
      */
40 40
     public function includeDevice(Port $port)
41 41
     {
Please login to merge, or discard this patch.
app/Http/Controllers/NotificationController.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     /**
45 45
      * Update a notifications status.
46 46
      *
47
-     * @return \Illuminate\Http\Response
47
+     * @return \Illuminate\Http\JsonResponse
48 48
      */
49 49
     public function update($id, $action)
50 50
     {
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
     /**
56 56
      * Create a new notification
57 57
      *
58
-     * @return \Illuminate\Http\Response
58
+     * @return \Illuminate\Http\JsonResponse
59 59
      */
60 60
     public function create(Request $request)
61 61
     {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
     public function index($type = null)
26 26
     {
27 27
 
28
-        $notifications = $this->api->be(auth()->user())->get('/api/notifications/'.$type);
28
+        $notifications = $this->api->be(auth()->user())->get('/api/notifications/' . $type);
29 29
         if ($type === 'archive')
30 30
         {
31 31
             $page   = '';
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
     public function update($id, $action)
50 50
     {
51 51
 
52
-        return response()->json($this->api->be(auth()->user())->patch('/api/notifications/'.$id.'/'.$action));
52
+        return response()->json($this->api->be(auth()->user())->patch('/api/notifications/' . $id . '/' . $action));
53 53
     }
54 54
 
55 55
     /**
Please login to merge, or discard this patch.
app/Models/Notification.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -42,6 +42,9 @@
 block discarded – undo
42 42
         $this->setAttrib('sticky', $enabled);
43 43
     }
44 44
 
45
+    /**
46
+     * @param string $name
47
+     */
45 48
     private function setAttrib($name, $enabled) {
46 49
         if ($enabled === true) {
47 50
             $read = new NotificationAttrib;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
 
71 71
     public function scopeLimit($query)
72 72
     {
73
-        return $query->select('notifications.*','key', 'users.username');
73
+        return $query->select('notifications.*', 'key', 'users.username');
74 74
     }
75 75
 
76 76
     public function scopeSource($query)
Please login to merge, or discard this patch.
app/Models/User.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
     /**
32 32
      * Test if the User is an admin or demo.
33 33
      *
34
-     * @return \Illuminate\Database\Eloquent\Builder
34
+     * @return boolean
35 35
      */
36 36
     public function isAdmin()
37 37
     {
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
      * Test if this user has global read access
43 43
      * these users have a level of 5, 10 or 11 (demo).
44 44
      *
45
-     * @return \Illuminate\Database\Eloquent\Builder
45
+     * @return boolean
46 46
      */
47 47
     public function hasGlobalRead()
48 48
     {
Please login to merge, or discard this patch.
app/Settings.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -220,7 +220,7 @@
 block discarded – undo
220 220
      * Clear the settings cache.
221 221
      * If path is set, only clear the path and it's parents.
222 222
      *
223
-     * @param null $key The path to clear.
223
+     * @param string $key The path to clear.
224 224
      */
225 225
     public function flush($key = null)
226 226
     {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@
 block discarded – undo
106 106
     public function get($key, $default = null)
107 107
     {
108 108
         // return value from cache or fetch it and return it
109
-        return Cache::tags(self::$cache_tag)->remember($key, $this->cache_time, function () use ($key, $default) {
109
+        return Cache::tags(self::$cache_tag)->remember($key, $this->cache_time, function() use ($key, $default) {
110 110
             // fetch the value from config.php first
111 111
             if (Config::has('config.' . $key)) {
112 112
                 $config_data = Config::get('config.' . $key, $default);
Please login to merge, or discard this patch.
app/Api/Controllers/APIController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
         $stats['ports']      = \App\Models\Port::all()->count();
36 36
         $stats['syslog']     = \App\Models\Syslog::all()->count();
37 37
         $stats['eventlog']   = \App\Models\Eventlog::all()->count();
38
-        $stats['apps']       = DB::table('applications')->count();;
38
+        $stats['apps']       = DB::table('applications')->count(); ;
39 39
         $stats['services']   = DB::table('services')->count();
40 40
         $stats['storage']    = DB::table('storage')->count();
41 41
         $stats['diskio']     = DB::table('ucd_diskio')->count();
Please login to merge, or discard this patch.
app/Api/Controllers/DeviceController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -67,11 +67,11 @@
 block discarded – undo
67 67
         }
68 68
         else {
69 69
             $user = User::find($request->user()->user_id);
70
-            $device =  $user->devices()->find($id);
70
+            $device = $user->devices()->find($id);
71 71
         }
72 72
         // morph the data as required
73 73
         if ($request->query('displayFormat') == 'link') {
74
-            return '<a href="'.url('/devices/').$device->deviceId.'">'.$device->hostname.'</a>';
74
+            return '<a href="' . url('/devices/') . $device->deviceId . '">' . $device->hostname . '</a>';
75 75
         }
76 76
 
77 77
         return $device;
Please login to merge, or discard this patch.
app/Api/Controllers/NotificationController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 
40 40
         $notification = Notification::find($id);
41 41
         $enable = strpos($action, 'un') === false;
42
-        if(!$enable) {
42
+        if (!$enable) {
43 43
             $action = substr($action, 2);
44 44
         }
45 45
 
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
         $notification = new Notification;
69 69
         $notification->title    = $request->title;
70 70
         $notification->body     = $request->body;
71
-        $notification->checksum = hash('sha512',$request->user()->user_id.'.LOCAL.'.$request->title);
71
+        $notification->checksum = hash('sha512', $request->user()->user_id . '.LOCAL.' . $request->title);
72 72
         $notification->source   = $request->user()->user_id;
73 73
         if ($notification->save())
74 74
         {
Please login to merge, or discard this patch.
app/Models/DbConfig.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -99,8 +99,8 @@
 block discarded – undo
99 99
         if (!empty($value)) {
100 100
             try {
101 101
                 return unserialize($value);
102
-            } catch(\Exception $e) {
103
-                if(starts_with($e->getMessage(), 'unserialize():')) {
102
+            } catch (\Exception $e) {
103
+                if (starts_with($e->getMessage(), 'unserialize():')) {
104 104
                     return $value;
105 105
                 } else {
106 106
                     throw $e;
Please login to merge, or discard this patch.