View GitHub Repository laravel-server-monitor-plugin-notification-by-host

Schedule Inspection
Subscribe
Passed
Push — master ( bcdbf3...5b3224 )
by Marc
11:54
created
Status
Category
Toggle Patches
src/Commands/ListNotifications.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -59,8 +59,7 @@  discard block
 
 block discarded – undo
59 59 
             $hostNotifications = $host->getCustomProperty('notifications');
60 60 
             if($hostNotifications) {
61 61 
                 $rows = $this->displayCustomNotification($hostNotifications, $rows);
62 
-            }
63 
-            else {
62 
+            } else {
64 63 
                 $rows = $this->displayGlobalNotification($rows);
65 64 
             }
66 65 
             $first = false;
 
@@ -80,12 +79,10 @@  discard block
 
 block discarded – undo
80 79 
                             $configurationList[] = $configurationAttribut.':'.(is_array($configurationValue)?implode(',', $configurationValue):$configurationValue);
81 80 
                         }
82 81 
                         $temp[] = implode("\n", $configurationList);
83 
-                    }
84 
-                    else {
82 
+                    } else {
85 83 
                         $temp[] = 'Global';
86 84 
                     }
87 
-                }
88 
-                else {
85 
+                } else {
89 86 
                     $temp[] = '';
90 87 
                 }
91 88 
             }
 
@@ -102,8 +99,7 @@  discard block
 
 block discarded – undo
102 99 
             foreach ($this->channels as $type => $channel) {
103 100 
                 if (array_search($type, $global) !== false) {
104 101 
                     $temp[] = 'Global';
105 
-                }
106 
-                else {
102 
+                } else {
107 103 
                     $temp[] = '';
108 104 
                 }
109 105 
             }
Please login to merge, or discard this patch.
src/Commands/AddNotificationByHost.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -50,8 +50,7 @@  discard block
 
 block discarded – undo
50 50 
                 if(!$configure[$channel]) {
51 51 
                     unset($configure[$channel]);
52 52 
                 }
53 
-            }
54 
-            else {
53 
+            } else {
55 54 
                 $this->error("No configuration for ".$channel." in server-monitor-plugin-notification-by-host.php file configuration");
56 55 
             }
57 56 
         }
 
@@ -99,8 +98,7 @@  discard block
 
 block discarded – undo
99 98 
     private function configuration($host, $notificationsParameters) {
100 99 
         if(!$notificationsParameters) {
101 100 
             $this->warn("The host {$host->name} has no notification configure !");
102 
-        }
103 
-        else {
101 
+        } else {
104 102
105 103 
             $this->line("----- Notification configuration -----");
106 104
Please login to merge, or discard this patch.