✨ Add fixer for AssignmentSpacingSniff

Author: Vincent Langlet

Symfony3Custom/Sniffs/WhiteSpace/AssignmentSpacingSniff.php

@@ -43,11 +43,20 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
                 || $tokens[$stackPtr + 1]['code'] !== T_WHITESPACE)
             && $tokens[$stackPtr - 1]['content'] !== 'strict_types'
         ) {
-            $phpcsFile->addError(
+            $fix = $phpcsFile->addFixableError(
                 'Add a single space around assignment operators',
                 $stackPtr,
                 'Invalid'
             );
+
+            if ($fix === true) {
+                if ($tokens[$stackPtr -1]['code'] !== T_WHITESPACE) {
+                    $phpcsFile->fixer->addContentBefore($stackPtr, ' ');
+                }
+                if ($tokens[$stackPtr +1]['code'] !== T_WHITESPACE) {
+                    $phpcsFile->fixer->addContent($stackPtr, ' ');
+                }
+            }
         }
     }
 }

Symfony3Custom/Tests/WhiteSpace/AssignmentSpacingUnitTest.inc

@@ -0,0 +1,15 @@
+<?php
+
+$a=1;
+$a = 1;
+
+$a+=1;
+$a += 1;
+
+$a.=1;
+$a .= 1;
+
+$arr = [
+    1=>1,
+    2 => 2,
+];

Symfony3Custom/Tests/WhiteSpace/AssignmentSpacingUnitTest.inc.fixed

@@ -0,0 +1,15 @@
+<?php
+
+$a = 1;
+$a = 1;
+
+$a += 1;
+$a += 1;
+
+$a .= 1;
+$a .= 1;
+
+$arr = [
+    1 => 1,
+    2 => 2,
+];

Symfony3Custom/Tests/WhiteSpace/AssignmentSpacingUnitTest.php

@@ -0,0 +1,42 @@
+<?php
+
+/**
+ * Unit test class for the AssignmentSpacing sniff.
+ *
+ * A sniff unit test checks a .inc file for expected violations of a single
+ * coding standard. Expected errors and warnings are stored in this class.
+ */
+class Symfony3Custom_Tests_WhiteSpace_AssignmentSpacingUnitTest
+    extends AbstractSniffUnitTest
+{
+    /**
+     * Returns the lines where errors should occur.
+     *
+     * The key of the array should represent the line number and the value
+     * should represent the number of errors that should occur on that line.
+     *
+     * @return array<int, int>
+     */
+    public function getErrorList()
+    {
+        return array(
+            3 => 1,
+            6 => 1,
+            9 => 1,
+            13 => 1,
+        );
+    }
+
+    /**
+     * Returns the lines where warnings should occur.
+     *
+     * The key of the array should represent the line number and the value
+     * should represent the number of warnings that should occur on that line.
+     *
+     * @return array(int => int)
+     */
+    protected function getWarningList()
+    {
+        return array();
+    }
+}