Completed
Pull Request — master (#4)
by Eric
02:14
created
src/Monitors/DiskUsageMonitor.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@
 block discarded – undo
101 101
     }
102 102
 
103 103
     /**
104
-     * @return mixed|string
104
+     * @return string
105 105
      */
106 106
     public function getPath()
107 107
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 
3 3
 namespace EricMakesStuff\ServerMonitor\Monitors;
4 4
 
5
-use Carbon\Carbon;
6 5
 use EricMakesStuff\ServerMonitor\Events\DiskUsageAlarm;
7 6
 use EricMakesStuff\ServerMonitor\Events\DiskUsageHealthy;
8 7
 use EricMakesStuff\ServerMonitor\Exceptions\InvalidPath;
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
      */
50 50
     public function runMonitor()
51 51
     {
52
-        if (! file_exists($this->path)) {
52
+        if (!file_exists($this->path)) {
53 53
             throw InvalidPath::pathDoesNotExist($this->path);
54 54
         }
55 55
 
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
         
60 60
         $this->usedSpace = $this->totalSpace - $this->freeSpace;
61 61
 
62
-        $this->percentageUsed = sprintf('%.2f',($this->usedSpace / $this->totalSpace) * 100);
62
+        $this->percentageUsed = sprintf('%.2f', ($this->usedSpace / $this->totalSpace) * 100);
63 63
         
64 64
         if ($this->percentageUsed >= $this->alarmPercentage) {
65 65
             event(new DiskUsageAlarm($this));
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
      */
98 98
     public function getPercentageUsed()
99 99
     {
100
-        return $this->percentageUsed . '%';
100
+        return $this->percentageUsed.'%';
101 101
     }
102 102
 
103 103
     /**
@@ -113,6 +113,6 @@  discard block
 block discarded – undo
113 113
      */
114 114
     public function getAlarmPercentage()
115 115
     {
116
-        return $this->alarmPercentage . '%';
116
+        return $this->alarmPercentage.'%';
117 117
     }
118 118
 }
Please login to merge, or discard this patch.
src/Notifications/Senders/Mail.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
 
29 29
     public function send()
30 30
     {
31
-        $this->mailer->raw($this->message, function (Message $message) {
31
+        $this->mailer->raw($this->message, function(Message $message) {
32 32
 
33 33
             $message
34 34
                 ->subject($this->subject)
Please login to merge, or discard this patch.
src/Commands/RunCommand.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
     {
22 22
         $monitors = ServerMonitorFactory::createForMonitorConfig(config('server-monitor.monitors'));
23 23
 
24
-        $monitors->each(function (BaseMonitor $monitor) {
24
+        $monitors->each(function(BaseMonitor $monitor) {
25 25
             $monitor->runMonitor();
26 26
         });
27 27
     }
Please login to merge, or discard this patch.
src/Monitors/HttpPingMonitor.php 2 patches
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -85,6 +85,10 @@
 block discarded – undo
85 85
         }
86 86
     }
87 87
 
88
+    /**
89
+     * @param string $html
90
+     * @param boolean|string $phrase
91
+     */
88 92
     protected function checkResponseContains($html, $phrase)
89 93
     {
90 94
         if (!$phrase) {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
             ]);
71 71
             $response = $guzzle->get($this->url);
72 72
             $this->responseCode = $response->getStatusCode();
73
-            $this->responseContent = (string)$response->getBody();
73
+            $this->responseContent = (string) $response->getBody();
74 74
         } catch (ClientException $e) {
75 75
             $response = $e->getResponse();
76 76
             $this->responseCode = $response->getStatusCode();
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
         }
79 79
 
80 80
         if ($this->responseCode != '200'
81
-            || ! $this->checkResponseContains($this->responseContent, $this->checkPhrase)) {
81
+            || !$this->checkResponseContains($this->responseContent, $this->checkPhrase)) {
82 82
             event(new HttpPingDown($this));
83 83
         } else {
84 84
             event(new HttpPingUp($this));
Please login to merge, or discard this patch.
src/Notifications/Notifier.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
     public function httpPingDown(HttpPingMonitor $httpPingMonitor)
72 72
     {
73 73
         $additionalInfo = '';
74
-        if ($httpPingMonitor->getCheckPhrase() && ! $httpPingMonitor->getResponseContainsPhrase()) {
74
+        if ($httpPingMonitor->getCheckPhrase() && !$httpPingMonitor->getResponseContainsPhrase()) {
75 75
             $additionalInfo = " Response did not contain \"{$httpPingMonitor->getCheckPhrase()}\"";
76 76
         }
77 77
 
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
94 94
         $senderNames = config("server-monitor.notifications.events.{$eventName}");
95 95
 
96 96
         collect($senderNames)
97
-            ->map(function ($senderName) {
97
+            ->map(function($senderName) {
98 98
                 $className = $senderName;
99 99
 
100 100
                 if (file_exists(__DIR__.'/Senders/'.ucfirst($senderName).'.php')) {
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
 
104 104
                 return app($className);
105 105
             })
106
-            ->each(function (SendsNotifications $sender) use ($subject, $message, $type) {
106
+            ->each(function(SendsNotifications $sender) use ($subject, $message, $type) {
107 107
                 try {
108 108
                     $sender
109 109
                         ->setSubject($subject)
Please login to merge, or discard this patch.
src/Monitors/SSLCertificateMonitor.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
         $this->processCertificate($this->certificateInfo);
65 65
 
66 66
         if ($this->certificateDaysUntilExpiration < 0
67
-            || ! $this->hostCoveredByCertificate($urlParts['host'], $this->certificateDomain, $this->certificateAdditionalDomains)) {
67
+            || !$this->hostCoveredByCertificate($urlParts['host'], $this->certificateDomain, $this->certificateAdditionalDomains)) {
68 68
             event(new SSLCertificateInvalid($this));
69 69
         } elseif (in_array($this->certificateDaysUntilExpiration, $this->alarmDaysBeforeExpiration)) {
70 70
             event(new SSLCertificateExpiring($this));
Please login to merge, or discard this patch.