Unverified Commit 5e23744d authored by Klaus Purer's avatar Klaus Purer Committed by GitHub
Browse files

feat(ScopeKeywordSpacing): Include scope keyword spacing sniff from Squiz (#3161389 by klausi)

parent e1d71c6b

coder_sniffer/Drupal/ruleset.xml

+1 −0
Original line number Diff line number Diff line
@@ -280,6 +280,7 @@ 
   <property name="ignoreNewlines" value="true"/>

  </properties>

 </rule>

 <rule ref="Squiz.WhiteSpace.ScopeKeywordSpacing" />

 <rule ref="Squiz.WhiteSpace.SemicolonSpacing" />

 <rule ref="Squiz.WhiteSpace.SuperfluousWhitespace" />

tests/Drupal/bad/BadUnitTest.php

+2 −1
Original line number Diff line number Diff line
@@ -376,7 +376,8 @@ class BadUnitTest extends CoderSniffUnitTest 
                827 => 1,

                829 => 1,

                836 => 1,

                838 => 2,

                846 => 2,

                852 => 2,

            ];

        }//end switch

tests/Drupal/bad/bad.php

+14 −0
Original line number Diff line number Diff line
@@ -835,4 +835,18 @@ function test28() { 
// Multiple statements on one line are not allowed.

echo 'Hi!';;



/**

 * A test class.

 */

class ScopeKeyword {



  /**

   * Much weird spacing here.

   */

  public  static  function test() {



  }



}



?>
 
 No newline at end of file

tests/Drupal/bad/bad.php.fixed

+14 −0
Original line number Diff line number Diff line
@@ -882,3 +882,17 @@ function test28() { 
// Multiple statements on one line are not allowed.

echo 'Hi!';

;



/**

 * A test class.

 */

class ScopeKeyword {



  /**

   * Much weird spacing here.

   */

  public static function test() {



  }



}