Squiz.WhiteSpace.LanguageConstructSpacing no longer checks for spaces…

… if parenthesis are being used (request squizlabs#1062)

CodeSniffer.php

@@ -73,7 +73,7 @@ class PHP_CodeSniffer
      *
      * @var string
      */
-    const VERSION = '2.6.3';
+    const VERSION = '2.7.0';
 
     /**
      * Package stability; either stable, beta or alpha.

CodeSniffer/Standards/Squiz/Sniffs/WhiteSpace/LanguageConstructSpacingSniff.php

@@ -82,7 +82,7 @@ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
                     $phpcsFile->fixer->replaceToken(($stackPtr + 1), ' ');
                 }
             }
-        } else {
+        } else if ($tokens[($stackPtr + 1)]['code'] !== T_OPEN_PARENTHESIS) {
             $error = 'Language constructs must be followed by a single space; expected "%s" but found "%s"';
             $data  = array(
                       $tokens[$stackPtr]['content'].' '.$tokens[($stackPtr + 1)]['content'],

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/LanguageConstructSpacingUnitTest.inc.fixed

@@ -0,0 +1,32 @@
+<?php
+echo $blah;
+echo $blah;
+echo($blah);
+
+print $blah;
+print $blah;
+print($blah);
+
+include $blah;
+include $blah;
+include($blah);
+
+include_once $blah;
+include_once $blah;
+include_once($blah);
+
+require $blah;
+require $blah;
+require($blah);
+
+require_once $blah;
+require_once $blah;
+require_once($blah);
+
+$obj = new MyClass();
+$obj = new MyClass();
+
+return;
+return $blah;
+return $blah;
+return($blah);

CodeSniffer/Standards/Squiz/Tests/WhiteSpace/LanguageConstructSpacingUnitTest.php

@@ -44,20 +44,13 @@ public function getErrorList()
     {
         return array(
                 3  => 1,
-                4  => 1,
                 7  => 1,
-                8  => 1,
                 11 => 1,
-                12 => 1,
                 15 => 1,
-                16 => 1,
                 19 => 1,
-                20 => 1,
                 23 => 1,
-                24 => 1,
                 27 => 1,
                 31 => 1,
-                32 => 1,
                );
 
     }//end getErrorList()

package.xml

@@ -17,15 +17,17 @@ http://pear.php.net/dtd/package-2.0.xsd">
  <date>2016-07-14</date>
  <time>09:28:00</time>
  <version>
-  <release>2.6.3</release>
-  <api>2.6.3</api>
+  <release>2.7.0</release>
+  <api>2.7.0</api>
  </version>
  <stability>
   <release>stable</release>
   <api>stable</api>
  </stability>
  <license uri="https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt">BSD 3-Clause License</license>
  <notes>
+  - Squiz.WhiteSpace.LanguageConstructSpacing no longer checks for spaces if parenthesis are being used (request #1062)
+    -- Makes this sniff more compatibile with those that check parenthesis spacing of function calls
   - Fixed bug #1066 : Undefined index: quiet in CLI.php during unit test run with -v command line arg
  </notes>
  <contents>
@@ -2137,6 +2139,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
          <tasks:replace from="@package_version@" to="version" type="package-info" />
         </file>
         <file baseinstalldir="PHP" name="LanguageConstructSpacingUnitTest.inc" role="test" />
+        <file baseinstalldir="PHP" name="LanguageConstructSpacingUnitTest.inc.fixed" role="test" />
         <file baseinstalldir="PHP" name="LanguageConstructSpacingUnitTest.php" role="test">
          <tasks:replace from="@package_version@" to="version" type="package-info" />
         </file>