[CodingStandard] first fixer fix?

deprecated-packages · May 6, 2017 · a32fba2 · a32fba2
1 parent 73cffaf
commit a32fba2
Showing 1 changed file with 36 additions and 15 deletions.
diff --git a/packages/CodingStandard/src/Fixer/Property/ArrayPropertyDefaultValueFixer.php b/packages/CodingStandard/src/Fixer/Property/ArrayPropertyDefaultValueFixer.php
@@ -3,23 +3,37 @@
 namespace Symplify\CodingStandard\Fixer\Property;
 
 use Nette\Utils\Strings;
-use PhpCsFixer\AbstractFixer;
 use PhpCsFixer\DocBlock\DocBlock;
+use PhpCsFixer\Fixer\FixerInterface;
 use PhpCsFixer\FixerDefinition\FixerDefinition;
 use PhpCsFixer\FixerDefinition\FixerDefinitionInterface;
+use PhpCsFixer\Tokenizer\CT;
 use PhpCsFixer\Tokenizer\Token;
 use PhpCsFixer\Tokenizer\Tokens;
 use SplFileInfo;
 
-final class ArrayPropertyDefaultValueFixer extends AbstractFixer
+final class ArrayPropertyDefaultValueFixer implements FixerInterface
 {
     public function isCandidate(Tokens $tokens): bool
     {
         // analyze only properties with comments
         return $tokens->isAllTokenKindsFound([T_DOC_COMMENT, T_VARIABLE]);
     }
 
-    protected function applyFix(SplFileInfo $file, Tokens $tokens): void
+    public function getDefinition(): FixerDefinitionInterface
+    {
+        return new FixerDefinition(
+            'Array property should have default value, to prevent undefined array issues.',
+            []
+        );
+    }
+
+    public function isRisky(): bool
+    {
+        return false;
+    }
+
+    public function fix(SplFileInfo $file, Tokens $tokens): void
     {
         foreach ($tokens as $index => $token) {
             if (! $token->isGivenKind(T_DOC_COMMENT)) {
@@ -42,12 +56,20 @@ protected function applyFix(SplFileInfo $file, Tokens $tokens): void
         }
     }
 
-    public function getDefinition(): FixerDefinitionInterface
+
+    public function getName(): string
     {
-        return new FixerDefinition(
-            'Array property should have default value, to prevent undefined array issues.',
-            []
-        );
+        return self::class;
+    }
+
+    public function getPriority(): int
+    {
+        return 0;
+    }
+
+    public function supports(SplFileInfo $file): bool
+    {
+        return true;
     }
 
     private function isArrayPropertyDocComment(Token $token): bool
@@ -70,13 +92,12 @@ private function isArrayPropertyDocComment(Token $token): bool
         return true;
     }
 
-    private function addDefaultValueForArrayProperty(Tokens $tokens,int $semicolonPosition): void
+    private function addDefaultValueForArrayProperty(Tokens $tokens, int $semicolonPosition): void
     {
-        $tokens->insertAt($semicolonPosition, new Token(']'));
-        $tokens->insertAt($semicolonPosition, new Token('['));
-        $tokens->insertAt($semicolonPosition, new Token(' '));
-        $tokens->insertAt($semicolonPosition, new Token('='));
-        $tokens->insertAt($semicolonPosition, new Token(' '));
+        $tokens->insertAt($semicolonPosition, new Token([CT::T_ARRAY_SQUARE_BRACE_CLOSE, ']']));
+        $tokens->insertAt($semicolonPosition, new Token([CT::T_ARRAY_SQUARE_BRACE_OPEN, '[']));
+        $tokens->insertAt($semicolonPosition, new Token([T_WHITESPACE, ' ']));
+        $tokens->insertAt($semicolonPosition, new Token([T_EQUAL, '=']));
+        $tokens->insertAt($semicolonPosition, new Token([T_WHITESPACE, ' ']));
     }
 }
-