phptek / silverstripe-sentry

src/Log/SentryLogger.php

Doc Comments +1 added lines, -1 removed lines

@@ -169,7 +169,7 @@
     /**
      * Return peak memory usage.
      *
-     * @return float
+     * @return string
      */
     public function getPeakMemory()
     {

Unused Use Statements +3 added lines, -3 removed lines

@@ -9,9 +9,9 @@
 
 namespace PhpTek\Sentry\Log;
 
-use SilverStripe\Control\Director,
-    SilverStripe\Core\Injector\Injector,
-    SilverStripe\Control\Middleware\TrustedProxyMiddleware;
+use SilverStripe\Control\Director;
+use SilverStripe\Core\Injector\Injector;
+use SilverStripe\Control\Middleware\TrustedProxyMiddleware;
 
 /**
  * The SentryLogWriter class simply acts as a bridge between the configured Sentry

Indentation +27 added lines, -27 removed lines

@@ -218,46 +218,46 @@
         return php_sapi_name();
     }
 
- 	/**
-	 * Returns the client IP address which originated this request.
-     * Lifted and modified from SilverStripe 3's SS_HTTPRequest.
-	 *
-	 * @return string
-	 */
-	public function getIP()
+        /**
+         * Returns the client IP address which originated this request.
+         * Lifted and modified from SilverStripe 3's SS_HTTPRequest.
+         *
+         * @return string
+         */
+    public function getIP()
     {
-		$headerOverrideIP = null;
+        $headerOverrideIP = null;
 
-		if (defined('TRUSTED_PROXY')) {
-			$headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
+        if (defined('TRUSTED_PROXY')) {
+            $headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
                 array(SS_TRUSTED_PROXY_IP_HEADER) :
                 null;
 
-			if(!$headers) {
-				// Backwards compatible defaults
-				$headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
-			}
+            if(!$headers) {
+                // Backwards compatible defaults
+                $headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
+            }
 
-			foreach($headers as $header) {
-				if(!empty($_SERVER[$header])) {
-					$headerOverrideIP = $_SERVER[$header];
+            foreach($headers as $header) {
+                if(!empty($_SERVER[$header])) {
+                    $headerOverrideIP = $_SERVER[$header];
 
-					break;
-				}
-			}
-		}
+                    break;
+                }
+            }
+        }
 
         $proxy = Injector::inst()->create(TrustedProxyMiddleware::class);
 
-		if ($headerOverrideIP) {
-			return $proxy->getIPFromHeaderValue($headerOverrideIP);
-		}
+        if ($headerOverrideIP) {
+            return $proxy->getIPFromHeaderValue($headerOverrideIP);
+        }
 
         if (isset($_SERVER['REMOTE_ADDR'])) {
-			return $_SERVER['REMOTE_ADDR'];
-		}
+            return $_SERVER['REMOTE_ADDR'];
+        }
 
         return '';
-	}
+    }
 
 }

Spacing +4 added lines, -5 removed lines

@@ -230,16 +230,15 @@
 
 		if (defined('TRUSTED_PROXY')) {
 			$headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
-                array(SS_TRUSTED_PROXY_IP_HEADER) :
-                null;
+                array(SS_TRUSTED_PROXY_IP_HEADER) : null;
 
-			if(!$headers) {
+			if (!$headers) {
 				// Backwards compatible defaults
 				$headers = ['HTTP_CLIENT_IP', 'HTTP_X_FORWARDED_FOR'];
 			}
 
-			foreach($headers as $header) {
-				if(!empty($_SERVER[$header])) {
+			foreach ($headers as $header) {
+				if (!empty($_SERVER[$header])) {
 					$headerOverrideIP = $_SERVER[$header];
 
 					break;

src/Adaptor/RavenClient.php

Spacing +2 added lines, -3 removed lines

@@ -81,7 +81,7 @@ 
      */
     public function setData($field, $data)
     {
-        switch($field) {
+        switch ($field) {
         case 'env':
             $this->client->setEnvironment($data);
             break;
@@ -121,8 +121,7 @@ 
     public function getLevel($level)
     {
         return isset($this->client->logLevels[$level]) ?
-            $this->client->logLevels[$level] :
-            $this->client->logLevels[self::$default_error_level];
+            $this->client->logLevels[$level] : $this->client->logLevels[self::$default_error_level];
     }
 
 }