Completed
Pull Request — master (#40)
by Simon
01:43
created
src/Adaptor/SentryAdaptor.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,6 @@
 block discarded – undo
14 14
 use Sentry\State\Scope;
15 15
 use Sentry\ClientInterface;
16 16
 use Sentry\Severity;
17
-use SilverStripe\Control\Director;
18 17
 use SilverStripe\Core\Config\Configurable;
19 18
 use SilverStripe\Core\Injector\Injector;
20 19
 use SilverStripe\Core\Environment as Env;
Please login to merge, or discard this patch.
src/Log/SentryLogger.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -372,7 +372,7 @@
 block discarded – undo
372 372
 
373 373
     /**
374 374
      * Check if we are on a tagged release and if so, use that to set the client release
375
-     * @param $ref
375
+     * @param string $ref
376 376
      * @return mixed
377 377
      */
378 378
     protected static function getGitTag($ref)
Please login to merge, or discard this patch.
Spacing   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
     {
184 184
         $peak = memory_get_peak_usage(true) / 1024 / 1024;
185 185
 
186
-        return (string)round($peak, 2) . 'Mb';
186
+        return (string) round($peak, 2) . 'Mb';
187 187
     }
188 188
 
189 189
     /**
@@ -238,8 +238,7 @@  discard block
 block discarded – undo
238 238
 
239 239
         if (defined('TRUSTED_PROXY')) {
240 240
             $headers = (defined('SS_TRUSTED_PROXY_IP_HEADER')) ?
241
-                [SS_TRUSTED_PROXY_IP_HEADER] :
242
-                null;
241
+                [SS_TRUSTED_PROXY_IP_HEADER] : null;
243 242
 
244 243
             if (!$headers) {
245 244
                 // Backwards compatible defaults
@@ -386,7 +385,7 @@  discard block
 block discarded – undo
386 385
                 }
387 386
                 $tags = explode(' ', $line);
388 387
                 if ($tags[0] === $ref) {
389
-                    list(,,$tag) = explode('/', $ref[1]);
388
+                    list(,, $tag) = explode('/', $ref[1]);
390 389
                     return $tag;
391 390
                 }
392 391
             }
Please login to merge, or discard this patch.