View GitHub Repository PHP_CodeSniffer

Schedule Inspection
Subscribe
Passed
Pull Request — master (#46)
by Björn
02:28
created
Status
Category
Toggle Patches
src/Standards/BestIt/Sniffs/Functions/MultipleReturnSniff.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -36,7 +36,7 @@  discard block
 
 block discarded – undo
36 36 
      *
37 37 
      * @var string
38 38 
      */
39 
-    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please ' .
39 
+    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please '.
40 40 
         'do not use an early return if your method/function still is cluttered.';
41 41
42 42 
     /**
 
@@ -65,7 +65,7 @@  discard block
 
 block discarded – undo
65 65 
             $this->token['scope_closer']
66 66 
         );
67 67
68 
-        return array_filter($returnPositions, function (int $returnPos): bool {
68 
+        return array_filter($returnPositions, function(int $returnPos): bool {
69 69 
             $possibleClosure = $this->file->findPrevious([T_CLOSURE, T_FUNCTION], $returnPos - 1, $this->stackPos);
70 70
71 71 
             return $possibleClosure === $this->stackPos;
 
@@ -84,7 +84,7 @@  discard block
 
 block discarded – undo
84 84 
         if (count($returnPositions) > 1) {
85 85 
             array_shift($returnPositions);
86 86
87 
-            array_walk($returnPositions, function (int $returnPos): void {
87 
+            array_walk($returnPositions, function(int $returnPos): void {
88 88 
                 $this->file->addWarning(
89 89 
                     self::WARNING_MULTIPLE_RETURNS_FOUND,
90 90 
                     $returnPos,
Please login to merge, or discard this patch.