jazzsequence / games-collector

vendor/wp-coding-standards/wpcs/WordPress/Tests/DB/RestrictedClassesUnitTest.3.inc

Braces +6 added lines, -3 removed lines

@@ -13,13 +13,16 @@
 		}
 	}
 	
-	class DBextended extends DBlayer { // Bad.
+	class DBextended extends DBlayer {
+// Bad.
 	}
 	
-	class DBextender implements PDO { // Ok -> resolves to \My\PDO.
+	class DBextender implements PDO {
+// Ok -> resolves to \My\PDO.
 	}
 	
-	class DBextendes implements \PDO { // Bad -> fully qualified as \PDO.
+	class DBextendes implements \PDO {
+// Bad -> fully qualified as \PDO.
 	}
 	
 	$db0 = new \DBlayer; // Ok - fully qualified as \DBlayer.

vendor/wp-coding-standards/wpcs/WordPress/Tests/CSRF/NonceVerificationUnitTest.inc

Spacing +21 added lines, -21 removed lines

@@ -2,24 +2,24 @@ 
 
 // Bad, needs nonce check.
 function bar() {
-	if ( ! isset( $_POST['test'] ) ) { // Bad.
+	if ( ! isset( $_POST[ 'test' ] ) ) { // Bad.
 		return;
 	}
 
-	do_something( $_POST['test'] ); // Bad.
+	do_something( $_POST[ 'test' ] ); // Bad.
 }
 
 // Good, has an nonce check.
 function ajax_process() {
 	check_ajax_referer( 'something' );
 
-	update_post_meta( (int) $_POST['id'], 'a_key', $_POST['a_value'] );
+	update_post_meta( (int) $_POST[ 'id' ], 'a_key', $_POST[ 'a_value' ] );
 }
 add_action( 'wp_ajax_process', 'ajax_process' );
 
 // It's also OK to check with isset() before the the nonce check.
 function foo() {
-	if ( ! isset( $_POST['test'] ) || ! wp_verify_nonce( 'some_action' ) ) {
+	if ( ! isset( $_POST[ 'test' ] ) || ! wp_verify_nonce( 'some_action' ) ) {
 		exit;
 	}
 
@@ -28,9 +28,9 @@ 
 
 // Doing other things with the request params before the nonce check is prohibited.
 function process() {
-	do_something( $_POST['foo'] ); // Bad.
+	do_something( $_POST[ 'foo' ] ); // Bad.
 
-	if ( ! isset( $_POST['test'] ) || ! wp_verify_nonce( 'some_action' ) ) {
+	if ( ! isset( $_POST[ 'test' ] ) || ! wp_verify_nonce( 'some_action' ) ) {
 		exit;
 	}
 
@@ -41,23 +41,23 @@ 
 
 	// Bad, needs nonce check.
 	function bar() {
-		if ( ! isset( $_POST['test'] ) ) { // Bad.
+		if ( ! isset( $_POST[ 'test' ] ) ) { // Bad.
 			return;
 		}
 
-		do_something( $_POST['test'] ); // Bad.
+		do_something( $_POST[ 'test' ] ); // Bad.
 	}
 
 	// Good, has an nonce check.
 	function ajax_process() {
 		check_ajax_referer( 'something' );
 
-		update_post_meta( (int) $_POST['id'], 'a_key', $_POST['a_value'] );
+		update_post_meta( (int) $_POST[ 'id' ], 'a_key', $_POST[ 'a_value' ] );
 	}
 
 	// It's also OK to check with isset() before the the nonce check.
 	function foo() {
-		if ( ! isset( $_POST['test'] ) || ! wp_verify_nonce( 'some_action' ) ) {
+		if ( ! isset( $_POST[ 'test' ] ) || ! wp_verify_nonce( 'some_action' ) ) {
 			exit;
 		}
 
@@ -66,9 +66,9 @@ 
 
 	// Doing other things with the request params before the nonce check is prohibited.
 	function process() {
-		do_something( $_POST['foo'] ); // Bad.
+		do_something( $_POST[ 'foo' ] ); // Bad.
 
-		if ( ! isset( $_POST['test'] ) || ! wp_verify_nonce( 'some_action' ) ) {
+		if ( ! isset( $_POST[ 'test' ] ) || ! wp_verify_nonce( 'some_action' ) ) {
 			exit;
 		}
 
@@ -79,19 +79,19 @@ 
 // Assignments are allowed.
 function foo_2() {
 	$_POST = array( 'a' => 'b' ); // OK.
-	$_POST['test'] = somethin(); // OK.
-	$_POST['settings'][ $setting ] = 'bb'; // OK.
+	$_POST[ 'test' ] = somethin(); // OK.
+	$_POST[ 'settings' ][ $setting ] = 'bb'; // OK.
 }
 
 // Particular cases can be whitelisted with a comment.
 function foo_3() {
-	bar( $_POST['var'] ); // WPCS: CSRF OK.
-	bar( $_POST['var'] ); // Bad.
+	bar( $_POST[ 'var' ] ); // WPCS: CSRF OK.
+	bar( $_POST[ 'var' ] ); // Bad.
 }
 
 // We need to account for when there are multiple vars in a single isset().
 function foo_4() {
-	if ( ! isset( $_POST['foo'], $_POST['bar'], $_POST['_wpnonce'] ) ) { // OK.
+	if ( ! isset( $_POST[ 'foo' ], $_POST[ 'bar' ], $_POST[ '_wpnonce' ] ) ) { // OK.
 		return;
 	}
 
@@ -101,8 +101,8 @@ 
 // Sanitization before the nonce check is permitted.
 function sanitization_allowed() {
 
-	$foo = (int) $_POST['foo']; // OK.
-	$bar = sanitize_key( $_POST['bar'] ); // OK.
+	$foo = (int) $_POST[ 'foo' ]; // OK.
+	$bar = sanitize_key( $_POST[ 'bar' ] ); // OK.
 
 	check_ajax_referer( "something-{$foo}-{$bar}" );
 }
@@ -110,8 +110,8 @@ 
 // The value must only be sanitized though.
 function foo_5() {
 
-	do_something( (int) $_POST['foo'] ); // Bad.
-	do_something( sanitize_key( $_POST['bar'] ) ); // Bad.
+	do_something( (int) $_POST[ 'foo' ] ); // Bad.
+	do_something( sanitize_key( $_POST[ 'bar' ] ) ); // Bad.
 
 	check_ajax_referer( 'something' );
 }

Braces +6 added lines, -3 removed lines

@@ -2,7 +2,8 @@ 
 
 // Bad, needs nonce check.
 function bar() {
-	if ( ! isset( $_POST['test'] ) ) { // Bad.
+	if ( ! isset( $_POST['test'] ) ) {
+// Bad.
 		return;
 	}
 
@@ -41,7 +42,8 @@ 
 
 	// Bad, needs nonce check.
 	function bar() {
-		if ( ! isset( $_POST['test'] ) ) { // Bad.
+		if ( ! isset( $_POST['test'] ) ) {
+// Bad.
 			return;
 		}
 
@@ -91,7 +93,8 @@ 
 
 // We need to account for when there are multiple vars in a single isset().
 function foo_4() {
-	if ( ! isset( $_POST['foo'], $_POST['bar'], $_POST['_wpnonce'] ) ) { // OK.
+	if ( ! isset( $_POST['foo'], $_POST['bar'], $_POST['_wpnonce'] ) ) {
+// OK.
 		return;
 	}
 

vendor/wp-coding-standards/wpcs/WordPress/Tests/NamingConventions/ValidHookNameUnitTest.1.inc

Spacing +2 added lines, -2 removed lines

@@ -6,12 +6,12 @@
 do_action( "admin_head-$hook_suffix" );
 do_action( 'admin_head-media-upload_popup' );
 apply_filters( "bulk_actions-{$this->screen->id}", $this->_actions );
-apply_filters( "current_theme-supports-{$feature}", true, $args, $_wp_theme_features[$feature] );
+apply_filters( "current_theme-supports-{$feature}", true, $args, $_wp_theme_features[ $feature ] );
 
 // These should still give warnings.
 do_action( "admin_head*$hook_suffix" ); // Warning - use underscore.
 do_action( 'admin_head.media.upload_popup' ); // Warning - use underscore.
 apply_filters( "bulk_actions {$this->screen->id}", $this->_actions ); // Warning - use underscore.
-apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[$feature] ); // Warning - use underscore.
+apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[ $feature ] ); // Warning - use underscore.
 
 // @codingStandardsChangeSetting WordPress.NamingConventions.ValidHookName additionalWordDelimiters _

vendor/wp-coding-standards/wpcs/WordPress/Tests/NamingConventions/ValidHookNameUnitTest.inc

Spacing +23 added lines, -23 removed lines

@@ -8,7 +8,7 @@ 
 do_action( "admin_head-$hook_suffix" ); // Warning - use underscore.
 do_action( 'admin_head.media.upload_popup' ); // Warning - use underscore.
 apply_filters( "bulk_actions {$this->screen->id}", $this->_actions ); // Warning - use underscore.
-apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[$feature] ); // Warning - use underscore.
+apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[ $feature ] ); // Warning - use underscore.
 
 // Simple strings.
 do_action( "adminHead" ); // Error - use lowercase.
@@ -22,11 +22,11 @@ 
 
 // Compound hook names.
 do_action( 'admin_head_' . $Type . '_action' ); // ok.
-do_action( 'admin_head_' .  get_ID() . '_action' ); // Ok.
+do_action( 'admin_head_' . get_ID() . '_action' ); // Ok.
 do_action( 'admin_head_' . $post->ID . '_action' ); // Ok.
 
 do_action( 'admin_Head_' . $Type . '_Action' ); // Error - use lowercase.
-do_action( 'admin_Head_' .  get_ID() . '_Action' ); // Error - use lowercase.
+do_action( 'admin_Head_' . get_ID() . '_Action' ); // Error - use lowercase.
 do_action( 'admin_Head_' . $post->ID . '_Action' ); // Error - use lowercase.
 
 do_action(
@@ -36,47 +36,47 @@ 
 
 // More complex strings.
 do_action( "admin_head_$Post" ); // Ok.
-do_action( "admin_head_$Post[1]_action" ); // Ok.
-do_action( "admin_head_$Post[Test]_action" ); // Ok.
+do_action( "admin_head_$Post[ 1 ]_action" ); // Ok.
+do_action( "admin_head_$Post[ Test ]_action" ); // Ok.
 do_action( "admin_head_${Post}_action" ); // Ok.
 do_action( "admin_head_$Post->ID" ); // Ok.
 do_action( "admin_head_{$Post}" ); // Ok.
-do_action( "admin_head_{$Post['Key']}_action" ); // Ok.
-do_action( "admin_head_{$Post[1][2]}_action" ); // Ok.
+do_action( "admin_head_{$Post[ 'Key' ]}_action" ); // Ok.
+do_action( "admin_head_{$Post[ 1 ][ 2 ]}_action" ); // Ok.
 do_action( "admin_head_{$post->ID}_action" ); // Ok.
-do_action( "admin_head_{$obj->Values[3]->name}_action" ); // Ok.
+do_action( "admin_head_{$obj->Values[ 3 ]->name}_action" ); // Ok.
 do_action( "admin_head_{${$Name}}_action" ); // Ok.
-do_action( "admin_head_{$foo->{$baz[1]}}_action" ); // Ok.
+do_action( "admin_head_{$foo->{$baz[ 1 ]}}_action" ); // Ok.
 do_action( "admin_head_{${getName()}}_action" ); // Ok.
 do_action( "admin_head_{${$object->getName()}}_action" ); // Ok.
 
 do_action( "admin_Head_$Post" ); // Error - use lowercase.
-do_action( "admin_Head_$Post[1]_Action" ); // Error - use lowercase.
-do_action( "admin_Head_$Post[Test]_Action" ); // Error - use lowercase.
+do_action( "admin_Head_$Post[ 1 ]_Action" ); // Error - use lowercase.
+do_action( "admin_Head_$Post[ Test ]_Action" ); // Error - use lowercase.
 do_action( "admin_Head_${Post}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_$Post->ID" ); // Error - use lowercase.
-do_action( "admin_Head_{$Post}" );  // Error - use lowercase.
-do_action( "admin_Head_{$Post['Key']}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$Post[1][2]}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post}" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post[ 'Key' ]}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post[ 1 ][ 2 ]}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{$post->ID}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$obj->Values[3]->name}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$obj->Values[ 3 ]->name}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{${$Name}}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$foo->{$baz[1]}}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$foo->{$baz[ 1 ]}}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{${getName()}}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{${$object->getName()}}_Action" ); // Error - use lowercase.
 
 do_action( "admin_Head_$Post admin_Head_$Post" ); // Error - use lowercase + warning about space.
-do_action( "admin_Head_$Post[1]_Action_$Post[1]_Action" ); // Error - use lowercase.
-do_action( "admin_Head_$Post[Test]_Action_$Post[Test]_Action" ); // Error - use lowercase.
+do_action( "admin_Head_$Post[ 1 ]_Action_$Post[ 1 ]_Action" ); // Error - use lowercase.
+do_action( "admin_Head_$Post[ Test ]_Action_$Post[ Test ]_Action" ); // Error - use lowercase.
 do_action( "admin_Head_${Post}_Action_${Post}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_$Post->ID admin_Head_$Post->ID" ); // Error - use lowercase + warning about space.
-do_action( "admin_Head_{$Post}_admin_Head_{$Post}" );  // Error - use lowercase.
-do_action( "admin_Head_{$Post['Key']}_Action_{$Post['Key']}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$Post[1][2]}_Action_{$Post[1][2]}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post}_admin_Head_{$Post}" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post[ 'Key' ]}_Action_{$Post[ 'Key' ]}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$Post[ 1 ][ 2 ]}_Action_{$Post[ 1 ][ 2 ]}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{$post->ID}_Action_{$post->ID}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$obj->Values[3]->name}-Action_{$obj->Values[3]->name}_Action" ); // Error - use lowercase + warning about dash.
+do_action( "admin_Head_{$obj->Values[ 3 ]->name}-Action_{$obj->Values[ 3 ]->name}_Action" ); // Error - use lowercase + warning about dash.
 do_action( "admin_Head_{${$Name}}_Action_{${$Name}}_Action" ); // Error - use lowercase.
-do_action( "admin_Head_{$foo->{$baz[1]}}_Action_{$foo->{$baz[1]}}_Action" ); // Error - use lowercase.
+do_action( "admin_Head_{$foo->{$baz[ 1 ]}}_Action_{$foo->{$baz[ 1 ]}}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{${getName()}}_Action_{${getName()}}_Action" ); // Error - use lowercase.
 do_action( "admin_Head_{${$object->getName()}}_Action_{${$object->getName()}}_Action" ); // Error - use lowercase.
 

vendor/wp-coding-standards/wpcs/WordPress/Tests/NamingConventions/ValidHookNameUnitTest.2.inc

Spacing +2 added lines, -2 removed lines

@@ -6,12 +6,12 @@
 do_action( "admin_head-$hook_suffix" );
 do_action( 'admin_head.media-upload_popup' );
 apply_filters( "bulk_actions-{$this->screen->id}", $this->_actions );
-apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[$feature] );
+apply_filters( "current_theme/supports-{$feature}", true, $args, $_wp_theme_features[ $feature ] );
 
 // These should still give warnings.
 do_action( "admin_head*$hook_suffix" ); // Warning - use underscore.
 do_action( 'admin_head&media+upload_popup' ); // Warning - use underscore.
 apply_filters( "bulk_actions {$this->screen->id}", $this->_actions ); // Warning - use underscore.
-apply_filters( "current_theme#supports-{$feature}", true, $args, $_wp_theme_features[$feature] ); // Warning - use underscore.
+apply_filters( "current_theme#supports-{$feature}", true, $args, $_wp_theme_features[ $feature ] ); // Warning - use underscore.
 
 // @codingStandardsChangeSetting WordPress.NamingConventions.ValidHookName additionalWordDelimiters _

vendor/wp-coding-standards/wpcs/WordPress/Tests/NamingConventions/ValidVariableNameUnitTest.inc

Spacing +15 added lines, -15 removed lines

@@ -37,16 +37,16 @@ 
 echo "Hello $varname";
 echo "Hello $_varName"; // Bad.
 
-echo 'Hello '.$varName; // Bad.
-echo 'Hello '.$var_name;
-echo 'Hello '.$varname;
-echo 'Hello '.$_varName; // Bad.
+echo 'Hello ' . $varName; // Bad.
+echo 'Hello ' . $var_name;
+echo 'Hello ' . $varname;
+echo 'Hello ' . $_varName; // Bad.
 
-echo $_SERVER['var_name'];
-echo $_REQUEST['var_name'];
-echo $_GET['var_name'];
-echo $_POST['var_name'];
-echo $GLOBALS['var_name'];
+echo $_SERVER[ 'var_name' ];
+echo $_REQUEST[ 'var_name' ];
+echo $_GET[ 'var_name' ];
+echo $_POST[ 'var_name' ];
+echo $GLOBALS[ 'var_name' ];
 
 echo MyClass::$varName; // Bad.
 echo MyClass::$var_name;
@@ -64,15 +64,15 @@ 
 echo $object_name->_varName2; // Bad.
 echo $object_name->VAR_name; // Bad.
 
-echo $this->myFunction($one, $two);
-echo $object->myFunction($one_two);
+echo $this->myFunction( $one, $two );
+echo $object->myFunction( $one_two );
 
 $error = "format is \$GLOBALS['$varName']"; // Bad.
 
-echo $_SESSION['var_name'];
-echo $_FILES['var_name'];
-echo $_ENV['var_name'];
-echo $_COOKIE['var_name'];
+echo $_SESSION[ 'var_name' ];
+echo $_FILES[ 'var_name' ];
+echo $_ENV[ 'var_name' ];
+echo $_COOKIE[ 'var_name' ];
 
 $XML       = 'hello'; // Bad.
 $myXML     = 'hello'; // Bad.

Braces +4 added lines, -2 removed lines

@@ -96,7 +96,8 @@ 
 	public $_public_leading_underscore;
 	private $private_no_underscore_loading;
 	
-	function Bar( $VARname ) { // Bad.
+	function Bar( $VARname ) {
+// Bad.
 		$localVariable = false; // Bad.
 		echo Some_Class::$VarName; // Bad.
 		echo $this->VAR_name; // Bad.
@@ -105,7 +106,8 @@ 
 		echo $this->_VAR_name; // Bad.
 	}
 
-	function Baz( $var_name ) { // Ok.
+	function Baz( $var_name ) {
+// Ok.
 		$local_variable = false; // Ok.
 		echo Some_Class::$var_name; // Ok.
 		echo $this->var_name; // Ok.

vendor/wp-coding-standards/wpcs/WordPress/Tests/Classes/ValidClassNameUnitTest.inc

Spacing +1 added lines, -1 removed lines

@@ -4,4 +4,4 @@
 
 class WpMyPluginName {} // Good.
 
-class wpMyPluginName{} // Bad, not in camel caps.
+class wpMyPluginName {} // Bad, not in camel caps.

vendor/wp-coding-standards/wpcs/WordPress/Tests/Classes/ClassOpeningStatementUnitTest.inc

Indentation +8 added lines, -8 removed lines

@@ -52,30 +52,30 @@
 
 // This is OK.
 class A_Class_With_Really_Long_Name
-    extends Another_Class_With_A_Really_Long_Name {
+	 extends Another_Class_With_A_Really_Long_Name {
 
 }
 
 // This is OK too.
 class A_Class_With_Really_Long_Name_2
-    extends Another_Class_With_A_Really_Long_Name
-    implements Some_Interface_With_A_Long_Name,
-          Another_Interface_With_A_Long_Name {
+	 extends Another_Class_With_A_Really_Long_Name
+	 implements Some_Interface_With_A_Long_Name,
+			 Another_Interface_With_A_Long_Name {
 
 }
 
 // But this is not.
 class A_Class_With_Really_Long_Name_3
-    extends Another_Class_With_A_Really_Long_Name
+	 extends Another_Class_With_A_Really_Long_Name
 	{
 
 }
 
 // Nor is this.
 class A_Class_With_Really_Long_Name_4
-    extends Another_Class_With_A_Really_Long_Name
-    implements Some_Interface_With_A_Long_Name,
-          Another_Interface_With_A_Long_Name
+	 extends Another_Class_With_A_Really_Long_Name
+	 implements Some_Interface_With_A_Long_Name,
+			 Another_Interface_With_A_Long_Name
 {
 
 }

Spacing +4 added lines, -4 removed lines

@@ -35,16 +35,16 @@
 }
 
 // These should all be flagged for wrong whitespace before opening brace.
-interface Test_Interface_Bad_C   {
+interface Test_Interface_Bad_C {
 }
 
-class Test_Class_Bad_G	{		
+class Test_Class_Bad_G {		
 }
 
-class Test_Class_Bad_H extends Test_Class_Bad_G			{
+class Test_Class_Bad_H extends Test_Class_Bad_G {
 }
 
-class Test_Class_Bad_I implements Test_Interface_Bad_C{
+class Test_Class_Bad_I implements Test_Interface_Bad_C {
 }
 
 // This one should be flagged as a potential parse error.

Braces +11 added lines, -17 removed lines

@@ -15,36 +15,31 @@ 
 
 
 // These are all incorrect.
-interface Test_Interface_Bad_A
-{ // There should be no content after the brace.
+interface Test_Interface_Bad_A {
+// There should be no content after the brace.
 }
 
-class Test_Class_Bad_A
-	{
+class Test_Class_Bad_A {
 	}
 
-class Test_Class_Bad_B extends Test_Class_Bad_A
-
-{ // There should be no content after the brace.
+class Test_Class_Bad_B extends Test_Class_Bad_A {
+// There should be no content after the brace.
 }
 
-class Test_Class_Bad_C implements Test_Interface_Bad_A
-
-
-{
+class Test_Class_Bad_C implements Test_Interface_Bad_A {
 }
 
 // These should all be flagged for wrong whitespace before opening brace.
-interface Test_Interface_Bad_C   {
+interface Test_Interface_Bad_C {
 }
 
-class Test_Class_Bad_G	{		
+class Test_Class_Bad_G {
 }
 
-class Test_Class_Bad_H extends Test_Class_Bad_G			{
+class Test_Class_Bad_H extends Test_Class_Bad_G {
 }
 
-class Test_Class_Bad_I implements Test_Interface_Bad_C{
+class Test_Class_Bad_I implements Test_Interface_Bad_C {
 }
 
 // This one should be flagged as a potential parse error.
@@ -66,8 +61,7 @@ 
 
 // But this is not.
 class A_Class_With_Really_Long_Name_3
-    extends Another_Class_With_A_Really_Long_Name
-	{
+    extends Another_Class_With_A_Really_Long_Name {
 
 }
 

vendor/wp-coding-standards/wpcs/WordPress/Tests/XSS/EscapeOutputUnitTest.inc

Indentation +14 added lines, -14 removed lines

@@ -22,13 +22,13 @@ 
 // Issue:#53.
 function custom_column_display( $column, $post_id )
 {
-    global $post;
-    switch ( $column ) {
-        case 'some_number' : 
-            echo (int) $test;
-            echo (int) get_post_meta( $post_id, SOME_NUMBER, true );
-        break;
-    }
+	 global $post;
+	 switch ( $column ) {
+		  case 'some_number' : 
+				echo (int) $test;
+				echo (int) get_post_meta( $post_id, SOME_NUMBER, true );
+		  break;
+	 }
 }
 
 
@@ -160,10 +160,10 @@ 
 esc_html_e( 'Something' ); // Ok.
 
 echo $something // Bad.
-     . esc_attr( 'baz-' // Rest is OK.
-	         . $r
-	         . ( $r === $active_round ? ' foo' : '' )
-	         . ( $r < $active_round ? ' bar' : '' )
+	  . esc_attr( 'baz-' // Rest is OK.
+				. $r
+				. ( $r === $active_round ? ' foo' : '' )
+				. ( $r < $active_round ? ' bar' : '' )
 	) . 'something';
 
 echo implode( '<br>', $items ); // Bad.
@@ -174,9 +174,9 @@ 
 echo join( '<br>', array_map( 'esc_html', $items ) ); // Ok.
 
 echo '<option name="' . esc_attr( $name ) . '"' .
-     ( $name === $selected ? ' selected' : '' ) .
-     '>' . esc_html( $value )
-     . '</option>';
+	  ( $name === $selected ? ' selected' : '' ) .
+	  '>' . esc_html( $value )
+	  . '</option>';
 
 _deprecated_hook( 'some_filter', '1.3.0', esc_html__( 'The $arg is deprecated.' ), 'some_other_filter' ); // Ok.
 _deprecated_hook( "filter_{$context}", '1.3.0', __( 'The $arg is deprecated.' ), sprintf( __( 'Some parsed message %s', $variable ) ) ); // Bad.

Spacing +5 added lines, -5 removed lines

@@ -34,12 +34,12 @@ 
 
 $foo = 'abc';
 echo $foo; // Bad, should have escaping function.
-echo 'Some Raw String';  // Good.
+echo 'Some Raw String'; // Good.
 
 echo '' . $bad; // Bad, should not validate.
 echo "this is $bad"; // Bad.
 echo esc_html( $good . $better ) . $foo; // Bad, should escape all concatenated elements.
-echo esc_html( $food . 'include'  ); // Good, eveything inside the escaping/sanitizing function should pass.
+echo esc_html( $food . 'include' ); // Good, eveything inside the escaping/sanitizing function should pass.
 echo esc_html( strtoupper( $ok ) ) . $foo; // Bad, again.
 echo esc_html( strtoupper( $ok ) ) . ' ' . esc_html( strtolower( $ok ) ); // Ok.
 
@@ -132,13 +132,13 @@ 
 _deprecated_function( __FUNCTION__, '1.3.0', esc_html( $another_func ) ); // Ok.
 _deprecated_file( __FILE__, '1.3.0' ); // Ok.
 _deprecated_argument( __METHOD__, '1.3.0', 'The $arg is deprecated.' ); // Ok.
-_doing_it_wrong( __METHOD__, "Invalid value for the 'bob' argument {$args['bob']}." ); // Bad.
-_doing_it_wrong( __METHOD__, "Invalid value for the 'bob' argument " . esc_html( $args['bob'] ) . "." ); // Ok.
+_doing_it_wrong( __METHOD__, "Invalid value for the 'bob' argument {$args[ 'bob' ]}." ); // Bad.
+_doing_it_wrong( __METHOD__, "Invalid value for the 'bob' argument " . esc_html( $args[ 'bob' ] ) . "." ); // Ok.
 
 trigger_error( "There was an error: {$message}", E_USER_NOTICE ); // Bad.
 trigger_error( "There was an error: " . esc_html( $message ), E_USER_NOTICE ); // Ok.
 
-echo '<p>' . sprintf( esc_html__( 'Some text -> %sLink text%s', 'textdomain' ), '<a href="' . esc_url( add_query_arg( array( 'page' => 'my_page' ), admin_url( 'admin.php' ) ) ) . '">', '</a>' ). '</p>'; // Ok.
+echo '<p>' . sprintf( esc_html__( 'Some text -> %sLink text%s', 'textdomain' ), '<a href="' . esc_url( add_query_arg( array( 'page' => 'my_page' ), admin_url( 'admin.php' ) ) ) . '">', '</a>' ) . '</p>'; // Ok.
 
 echo '<br/><strong>' . sprintf( esc_html__( 'Found %d results', 'textdomain' ), (int) $result_count ) . '</strong><br/><br/>'; // Ok.
 

Braces +1 added lines, -2 removed lines

@@ -20,8 +20,7 @@
 
 <?php
 // Issue:#53.
-function custom_column_display( $column, $post_id )
-{
+function custom_column_display( $column, $post_id ) {
     global $post;
     switch ( $column ) {
         case 'some_number' :