Changeset 42343 for trunk/tests/phpunit/includes/functions.php

Timestamp:

11/30/2017 11:09:33 PM (7 years ago)

Author:

pento

Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:

• trunk/tests/phpunit/includes/functions.php (modified) (10 diffs)

trunk/tests/phpunit/includes/functions.php

@@ -21 +21 @@
 
 // For adding hooks before loading WP
-function tests_add_filter($tag, $function_to_add, $priority = 10, $accepted_args = 1) {
+function tests_add_filter( $tag, $function_to_add, $priority = 10, $accepted_args = 1 ) {
     global $wp_filter;
 
@@ -27 +27 @@
         add_filter( $tag, $function_to_add, $priority, $accepted_args );
     } else {
-        $idx = _test_filter_build_unique_id($tag, $function_to_add, $priority);
-        $wp_filter[$tag][$priority][$idx] = array('function' => $function_to_add, 'accepted_args' => $accepted_args);
+        $idx                                    = _test_filter_build_unique_id( $tag, $function_to_add, $priority );
+        $wp_filter[ $tag ][ $priority ][ $idx ] = array(
+            'function'      => $function_to_add,
+            'accepted_args' => $accepted_args,
+        );
     }
     return true;
 }
 
-function _test_filter_build_unique_id($tag, $function, $priority) {
-    if ( is_string($function) )
+function _test_filter_build_unique_id( $tag, $function, $priority ) {
+    if ( is_string( $function ) ) {
         return $function;
+    }
 
-    if ( is_object($function) ) {
+    if ( is_object( $function ) ) {
         // Closures are currently implemented as objects
         $function = array( $function, '' );
@@ -44 +48 @@
     }
 
-    if (is_object($function[0]) ) {
-        return spl_object_hash($function[0]) . $function[1];
-    } else if ( is_string($function[0]) ) {
+    if ( is_object( $function[0] ) ) {
+        return spl_object_hash( $function[0] ) . $function[1];
+    } elseif ( is_string( $function[0] ) ) {
         // Static Calling
-        return $function[0].$function[1];
+        return $function[0] . $function[1];
     }
 }
@@ -61 +65 @@
         $wpdb->commentmeta,
         $wpdb->term_relationships,
-        $wpdb->termmeta
+        $wpdb->termmeta,
     ) as $table ) {
         $wpdb->query( "DELETE FROM {$table}" );
@@ -68 +72 @@
     foreach ( array(
         $wpdb->terms,
-        $wpdb->term_taxonomy
+        $wpdb->term_taxonomy,
     ) as $table ) {
         $wpdb->query( "DELETE FROM {$table} WHERE term_id != 1" );
@@ -97 +101 @@
 
 function _wp_die_handler( $message, $title = '', $args = array() ) {
-    if ( !$GLOBALS['_wp_die_disabled'] ) {
-        _wp_die_handler_txt( $message, $title, $args);
+    if ( ! $GLOBALS['_wp_die_disabled'] ) {
+        _wp_die_handler_txt( $message, $title, $args );
     } else {
         //Ignore at our peril
@@ -126 +130 @@
     if ( ! empty( $args ) ) {
         echo "Args: \n";
-        foreach( $args as $k => $v ){
+        foreach ( $args as $k => $v ) {
             echo "\t $k : $v\n";
         }
@@ -138 +142 @@
     if ( ! empty( $args ) ) {
         echo "Args: \n";
-        foreach( $args as $k => $v ){
+        foreach ( $args as $k => $v ) {
             echo "\t $k : $v\n";
         }
@@ -164 +168 @@
 
     $uploads['subdir'] = '';
-    $uploads['path'] = str_replace( $subdir, '', $uploads['path'] );
-    $uploads['url'] = str_replace( $subdir, '', $uploads['url'] );
+    $uploads['path']   = str_replace( $subdir, '', $uploads['path'] );
+    $uploads['url']    = str_replace( $subdir, '', $uploads['url'] );
 
     return $uploads;
@@ -174 +178 @@
  */
 function _upload_dir_https( $uploads ) {
-    $uploads['url'] = str_replace( 'http://', 'https://', $uploads['url'] );
+    $uploads['url']     = str_replace( 'http://', 'https://', $uploads['url'] );
     $uploads['baseurl'] = str_replace( 'http://', 'https://', $uploads['baseurl'] );