📦 Use already existing sniff

Author: VincentLanglet

SymfonyCustom/Sniffs/WhiteSpace/CloseBracketSpacingSniff.php

@@ -9,7 +9,7 @@
 use PHP_CodeSniffer\Util\Tokens;
 
 /**
- * Checks that there is no white space before a closing bracket, for ")", "}", and array bracket.
+ * Checks that there is no white space before a closing array bracket.
  * Square Brackets are handled by Squiz\Sniffs\Arrays\ArrayBracketSpacingSniff.
  */
 class CloseBracketSpacingSniff implements Sniff
@@ -19,7 +19,7 @@ class CloseBracketSpacingSniff implements Sniff
      */
     public function register(): array
     {
-        return [T_CLOSE_CURLY_BRACKET, T_CLOSE_PARENTHESIS, T_CLOSE_SHORT_ARRAY];
+        return [T_CLOSE_SHORT_ARRAY];
     }
 
     /**

SymfonyCustom/Sniffs/WhiteSpace/OpenBracketSpacingSniff.php

@@ -8,7 +8,7 @@
 use PHP_CodeSniffer\Sniffs\Sniff;
 
 /**
- * Checks that there is no white space after an opening bracket, for "(", "{", and array bracket.
+ * Checks that there is no white space after an opening array bracket.
  * Square Brackets are handled by Squiz_Sniffs_Arrays_ArrayBracketSpacingSniff.
  */
 class OpenBracketSpacingSniff implements Sniff
@@ -18,7 +18,7 @@ class OpenBracketSpacingSniff implements Sniff
      */
     public function register(): array
     {
-        return [T_OPEN_CURLY_BRACKET, T_OPEN_PARENTHESIS, T_OPEN_SHORT_ARRAY];
+        return [T_OPEN_SHORT_ARRAY];
     }
 
     /**

SymfonyCustom/ruleset.xml

@@ -59,6 +59,11 @@
         </properties>
     </rule>
     <rule ref="Generic.WhiteSpace.SpreadOperatorSpacingAfter"/>
+    <rule ref="Generic.WhiteSpace.ArbitraryParenthesesSpacing">
+        <properties>
+            <property name="ignoreNewlines" value="true" />
+        </properties>
+    </rule>
     <rule ref="Squiz.Arrays.ArrayBracketSpacing"/>
     <rule ref="Squiz.Strings.ConcatenationSpacing">
         <properties>