Completed
Pull Request — master (#59)
by
unknown
03:17
created
src/Standards/BestIt/Sniffs/DocTags/ReturnTagSniff.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
     /**
36 36
      * The message for the mixed type warning.
37 37
      */
38
-    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the ' .
38
+    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the '.
39 39
         'required types in detail.';
40 40
 
41 41
     /**
Please login to merge, or discard this patch.
src/Standards/BestIt/Sniffs/DocTags/DeprecatedTagSniff.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
22 22
     /**
23 23
      * The message for the error.
24 24
      */
25
-    private const MESSAGE_TAG_MISSING_DATES = 'Please provide the version since when its deprecated and when it will ' .
25
+    private const MESSAGE_TAG_MISSING_DATES = 'Please provide the version since when its deprecated and when it will '.
26 26
         'be removed (Pattern: %s).';
27 27
 
28 28
     /**
Please login to merge, or discard this patch.
src/Standards/BestIt/Sniffs/DocTags/ParamTagSniff.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     /**
75 75
      * The message for the mixed type warning.
76 76
      */
77
-    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the ' .
77
+    private const MESSAGE_TAG_MIXED_TYPE = 'We suggest that you avoid the "mixed" type and declare the '.
78 78
         'required types in detail.';
79 79
 
80 80
     /**
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
     {
198 198
         $varOfThisTag = $this->getArgumentTokenOfTag();
199 199
 
200
-        return '/(?P<type>[\w|\|\[\]]*) ?(?P<var>' . preg_quote($varOfThisTag['content'], '/') .
200
+        return '/(?P<type>[\w|\|\[\]]*) ?(?P<var>'.preg_quote($varOfThisTag['content'], '/').
201 201
             ') ?(?P<desc>.*)/m';
202 202
     }
203 203
 
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
             ))->setToken($this->token);
235 235
         }
236 236
 
237
-        $this->varTokens = array_filter($this->tokens, function (array $token) use ($varPositions): bool {
237
+        $this->varTokens = array_filter($this->tokens, function(array $token) use ($varPositions): bool {
238 238
             return in_array($token['pointer'], $varPositions, true);
239 239
         });
240 240
 
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
             $this->stackPos - 1
257 257
         );
258 258
 
259
-        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function (int $position) {
259
+        $tagPosBeforeThis = array_filter($tagPosBeforeThis, function(int $position) {
260 260
             return $this->tokens[$position]['content'] === '@param';
261 261
         });
262 262
 
Please login to merge, or discard this patch.
src/Standards/BestIt/Sniffs/Functions/MultipleReturnSniff.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     /**
33 33
      * Error message for multiple returns.
34 34
      */
35
-    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please ' .
35
+    private const WARNING_MULTIPLE_RETURNS_FOUND = 'Multiple returns detected. Did you refactor your method? Please '.
36 36
         'do not use an early return if your method/function still is cluttered.';
37 37
 
38 38
     /**
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
             $this->token['scope_closer']
62 62
         );
63 63
 
64
-        return array_filter($returnPositions, function (int $returnPos): bool {
64
+        return array_filter($returnPositions, function(int $returnPos): bool {
65 65
             $possibleClosure = $this->file->findPrevious([T_CLOSURE, T_FUNCTION], $returnPos - 1, $this->stackPos);
66 66
 
67 67
             return $possibleClosure === $this->stackPos;
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         if (count($returnPositions) > 1) {
81 81
             array_shift($returnPositions);
82 82
 
83
-            array_walk($returnPositions, function (int $returnPos): void {
83
+            array_walk($returnPositions, function(int $returnPos): void {
84 84
                 $this->file->addWarning(
85 85
                     self::WARNING_MULTIPLE_RETURNS_FOUND,
86 86
                     $returnPos,
Please login to merge, or discard this patch.