Completed
Pull Request — master (#9)
by Thomas
01:34
created
src/Adaptor/RavenClient.php 1 patch
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.
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.
Indentation   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -218,46 +218,46 @@
 block discarded – undo
218 218
         return php_sapi_name();
219 219
     }
220 220
 
221
- 	/**
222
-	 * Returns the client IP address which originated this request.
223
-     * Lifted and modified from SilverStripe 3's SS_HTTPRequest.
224
-	 *
225
-	 * @return string
226
-	 */
227
-	public function getIP()
221
+        /**
222
+         * Returns the client IP address which originated this request.
223
+         * Lifted and modified from SilverStripe 3's SS_HTTPRequest.
224
+         *
225
+         * @return string
226
+         */
227
+    public function getIP()
228 228
     {
229
-		$headerOverrideIP = null;
229
+        $headerOverrideIP = null;
230 230
 
231
-		if (defined('TRUSTED_PROXY')) {
232
-			$headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
231
+        if (defined('TRUSTED_PROXY')) {
232
+            $headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
233 233
                 array(SS_TRUSTED_PROXY_IP_HEADER) :
234 234
                 null;
235 235
 
236
-			if(!$headers) {
237
-				// Backwards compatible defaults
238
-				$headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
239
-			}
236
+            if(!$headers) {
237
+                // Backwards compatible defaults
238
+                $headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
239
+            }
240 240
 
241
-			foreach($headers as $header) {
242
-				if(!empty($_SERVER[$header])) {
243
-					$headerOverrideIP = $_SERVER[$header];
241
+            foreach($headers as $header) {
242
+                if(!empty($_SERVER[$header])) {
243
+                    $headerOverrideIP = $_SERVER[$header];
244 244
 
245
-					break;
246
-				}
247
-			}
248
-		}
245
+                    break;
246
+                }
247
+            }
248
+        }
249 249
 
250 250
         $proxy = Injector::inst()->create(TrustedProxyMiddleware::class);
251 251
 
252
-		if ($headerOverrideIP) {
253
-			return $proxy->getIPFromHeaderValue($headerOverrideIP);
254
-		}
252
+        if ($headerOverrideIP) {
253
+            return $proxy->getIPFromHeaderValue($headerOverrideIP);
254
+        }
255 255
 
256 256
         if (isset($_SERVER['REMOTE_ADDR'])) {
257
-			return $_SERVER['REMOTE_ADDR'];
258
-		}
257
+            return $_SERVER['REMOTE_ADDR'];
258
+        }
259 259
 
260 260
         return '';
261
-	}
261
+    }
262 262
 
263 263
 }
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.