Merge branch 'fixArbitrarySpacing' of https://github.com/VincentLanglet/PHP_CodeSniffer

Author: gsherwood

src/Standards/Generic/Sniffs/WhiteSpace/ArbitraryParenthesesSpacingSniff.php

@@ -102,7 +102,10 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         $preOpener = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($opener - 1), null, true);
-        if ($preOpener !== false && isset($this->ignoreTokens[$tokens[$preOpener]['code']]) === true) {
+        if ($preOpener !== false
+            && isset($this->ignoreTokens[$tokens[$preOpener]['code']]) === true
+            && isset($tokens[$preOpener]['scope_condition']) === false
+        ) {
             // Function or language construct call.
             return;
         }

src/Standards/Generic/Tests/WhiteSpace/ArbitraryParenthesesSpacingUnitTest.inc

@@ -159,3 +159,7 @@ $a = (
     null !== $extra
 );
 // phpcs:set Generic.WhiteSpace.ArbitraryParenthesesSpacing ignoreNewlines false
+
+if (true) {} ( 1+2) === 3 ? $a = 1 : $a = 2;
+class A {} ( 1+2) === 3 ? $a = 1 : $a = 2;
+function foo() {} ( 1+2) === 3 ? $a = 1 : $a = 2;

src/Standards/Generic/Tests/WhiteSpace/ArbitraryParenthesesSpacingUnitTest.inc.fixed

@@ -147,3 +147,7 @@ $a = (
     null !== $extra
 );
 // phpcs:set Generic.WhiteSpace.ArbitraryParenthesesSpacing ignoreNewlines false
+
+if (true) {} (1+2) === 3 ? $a = 1 : $a = 2;
+class A {} (1+2) === 3 ? $a = 1 : $a = 2;
+function foo() {} (1+2) === 3 ? $a = 1 : $a = 2;

src/Standards/Generic/Tests/WhiteSpace/ArbitraryParenthesesSpacingUnitTest.php

@@ -56,6 +56,9 @@ public function getErrorList()
             141 => 1,
             144 => 1,
             146 => 1,
+            163 => 1,
+            164 => 1,
+            165 => 1,
         ];
 
     }//end getErrorList()