phptek / silverstripe-sentry

src/Adaptor/RavenClient.php

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

@@ -9,9 +9,9 @@
 
 namespace PhpTek\Sentry\Adaptor;
 
-use PhpTek\Sentry\Adaptor\SentryClientAdaptor,
-    phpTek\Sentry\Exception\SentryLogWriterException,
-    SilverStripe\Core\Config\Config;
+use PhpTek\Sentry\Adaptor\SentryClientAdaptor;
+use phpTek\Sentry\Exception\SentryLogWriterException;
+use SilverStripe\Core\Config\Config;
 
 /**
  * The RavenClient class simply acts as a bridge between the Raven PHP SDK and

Switch Indentation +15 added lines, -15 removed lines

@@ -82,21 +82,21 @@
     public function setData($field, $data)
     {
         switch($field) {
-        case 'env':
-            $this->client->setEnvironment($data);
-            break;
-        case 'tags':
-            $this->client->tags_context($data);
-            break;
-        case 'user':
-            $this->client->user_context($data);
-            break;
-        case 'extra':
-            $this->client->extra_context($data);
-            break;
-        default:
-            $msg = sprintf('Unknown field "%s" passed to %s().', $field, __FUNCTION__);
-            throw new SentryLogWriterException($msg);
+            case 'env':
+                $this->client->setEnvironment($data);
+                break;
+            case 'tags':
+                $this->client->tags_context($data);
+                break;
+            case 'user':
+                $this->client->user_context($data);
+                break;
+            case 'extra':
+                $this->client->extra_context($data);
+                break;
+            default:
+                $msg = sprintf('Unknown field "%s" passed to %s().', $field, __FUNCTION__);
+                throw new SentryLogWriterException($msg);
         }
     }
     

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];
     }
 
 }

src/Handler/SentryMonologHandler.php

Doc Comments +1 added lines

@@ -126,6 +126,7 @@
      * the request.
      * 
      * @param  Member $member
+     * @param SentryLogger $logger
      * @return array
      */
     private function getUserData(Member $member = null, $logger)

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

@@ -9,11 +9,11 @@
 
 namespace PhpTek\Sentry\Handler;
 
-use PhpTek\Sentry\Monolog\Handler\SentryRavenHandler,
-    Monolog\Logger,
-    SilverStripe\Dev\Backtrace,
-    SilverStripe\Security\Member,
-    PhpTek\Sentry\Log\SentryLogger;
+use PhpTek\Sentry\Monolog\Handler\SentryRavenHandler;
+use Monolog\Logger;
+use SilverStripe\Dev\Backtrace;
+use SilverStripe\Security\Member;
+use PhpTek\Sentry\Log\SentryLogger;
 
 /**
  * Monolog Handler for Sentry via Raven

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/Handler/SentryRavenHandler.php

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

@@ -13,9 +13,9 @@
 
 namespace PhpTek\Sentry\Monolog\Handler;
 
-use Monolog\Handler\RavenHandler,
-    Monolog\Logger,
-    Raven_Client;
+use Monolog\Handler\RavenHandler;
+use Monolog\Logger;
+use Raven_Client;
 
 /**
  * Subclasses RavenHandler purely to overload its `write()` method.