Completed
Pull Request — master (#8)
by Thomas
07:46
created
src/Adaptor/RavenClient.php 2 patches
Unused Use Statements   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -9,9 +9,9 @@
 block discarded – undo
9 9
 
10 10
 namespace PhpTek\Sentry\Adaptor;
11 11
 
12
-use PhpTek\Sentry\Adaptor\SentryClientAdaptor,
13
-    phpTek\Sentry\Exception\SentryLogWriterException,
14
-    SilverStripe\Core\Config\Config;
12
+use PhpTek\Sentry\Adaptor\SentryClientAdaptor;
13
+use phpTek\Sentry\Exception\SentryLogWriterException;
14
+use SilverStripe\Core\Config\Config;
15 15
 
16 16
 /**
17 17
  * The RavenClient class simply acts as a bridge between the Raven PHP SDK and
Please login to merge, or discard this patch.
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
      */
82 82
     public function setData($field, $data)
83 83
     {
84
-        switch($field) {
84
+        switch ($field) {
85 85
         case 'env':
86 86
             $this->client->setEnvironment($data);
87 87
             break;
@@ -121,8 +121,7 @@  discard block
 block discarded – undo
121 121
     public function getLevel($level)
122 122
     {
123 123
         return isset($this->client->logLevels[$level]) ?
124
-            $this->client->logLevels[$level] :
125
-            $this->client->logLevels[self::$default_error_level];
124
+            $this->client->logLevels[$level] : $this->client->logLevels[self::$default_error_level];
126 125
     }
127 126
 
128 127
 }
Please login to merge, or discard this patch.
src/Log/SentryLogger.php 2 patches
Unused Use Statements   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -9,9 +9,9 @@
 block discarded – undo
9 9
 
10 10
 namespace PhpTek\Sentry\Log;
11 11
 
12
-use SilverStripe\Control\Director,
13
-    SilverStripe\Core\Injector\Injector,
14
-    SilverStripe\Control\Middleware\TrustedProxyMiddleware;
12
+use SilverStripe\Control\Director;
13
+use SilverStripe\Core\Injector\Injector;
14
+use SilverStripe\Control\Middleware\TrustedProxyMiddleware;
15 15
 
16 16
 /**
17 17
  * The SentryLogWriter class simply acts as a bridge between the configured Sentry
Please login to merge, or discard this patch.
Spacing   +4 added lines, -5 removed lines patch added patch discarded remove patch
@@ -230,16 +230,15 @@
 block discarded – undo
230 230
 
231 231
 		if (defined('TRUSTED_PROXY')) {
232 232
 			$headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
233
-                array(SS_TRUSTED_PROXY_IP_HEADER) :
234
-                null;
233
+                array(SS_TRUSTED_PROXY_IP_HEADER) : null;
235 234
 
236
-			if(!$headers) {
235
+			if (!$headers) {
237 236
 				// Backwards compatible defaults
238 237
 				$headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
239 238
 			}
240 239
 
241
-			foreach($headers as $header) {
242
-				if(!empty($_SERVER[$header])) {
240
+			foreach ($headers as $header) {
241
+				if (!empty($_SERVER[$header])) {
243 242
 					$headerOverrideIP = $_SERVER[$header];
244 243
 
245 244
 					break;
Please login to merge, or discard this patch.
src/Handler/SentryMonologHandler.php 1 patch
Unused Use Statements   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -9,11 +9,11 @@
 block discarded – undo
9 9
 
10 10
 namespace PhpTek\Sentry\Handler;
11 11
 
12
-use PhpTek\Sentry\Monolog\Handler\SentryRavenHandler,
13
-    Monolog\Logger,
14
-    SilverStripe\Dev\Backtrace,
15
-    SilverStripe\Security\Member,
16
-    PhpTek\Sentry\Log\SentryLogger;
12
+use PhpTek\Sentry\Monolog\Handler\SentryRavenHandler;
13
+use Monolog\Logger;
14
+use SilverStripe\Dev\Backtrace;
15
+use SilverStripe\Security\Member;
16
+use PhpTek\Sentry\Log\SentryLogger;
17 17
 
18 18
 /**
19 19
  * Monolog Handler for Sentry via Raven
Please login to merge, or discard this patch.
src/Handler/SentryRavenHandler.php 1 patch
Unused Use Statements   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -13,9 +13,9 @@
 block discarded – undo
13 13
 
14 14
 namespace PhpTek\Sentry\Monolog\Handler;
15 15
 
16
-use Monolog\Handler\RavenHandler,
17
-    Monolog\Logger,
18
-    Raven_Client;
16
+use Monolog\Handler\RavenHandler;
17
+use Monolog\Logger;
18
+use Raven_Client;
19 19
 
20 20
 /**
21 21
  * Subclasses RavenHandler purely to overload its `write()` method.
Please login to merge, or discard this patch.