Changeset 4906 for sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/wp-cli-commands/rewrite-rules.php

Timestamp:

02/15/2017 02:32:16 PM (8 years ago)

Author:

iandunn

Message:

WordCamp WP-CLI: Replace success messages with progress meter

Outputting a success message for each site clutters the screen, making it hard to scan warnings. A progress meter is a better way to provide feedback to the user.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/wp-cli-commands/rewrite-rules.php (modified) (2 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/mu-plugins/wp-cli-commands/rewrite-rules.php

@@ -21 +21 @@
         $error           = '';
         $sites           = wp_get_sites( array( 'limit' => false ) );
+        $notify          = new \cli\progress\Bar( sprintf( 'Processing %d sites', count( $sites ) ), count( $sites ) );
 
         WP_CLI::line();
@@ -53 +54 @@
             }
 
-            if ( $success ) {
-                WP_CLI::line( sprintf( '%s: Flushed', $display_url ) );
-            } else {
+            if ( ! $success ) {
                 WP_CLI::warning( sprintf( '%s: Failed with error: %s', $display_url, $error ) );
             }
+
+            $notify->tick();
         }
+
+        $notify->finish();
 
         $execution_time = microtime( true ) - $start_timestamp;