Changeset 3053

Timestamp:

04/30/2016 09:56:34 AM (10 years ago)

Author:

ocean90

Message:

Translate, Plugin Directory: Use WP_CLI::error() instead of WP_CLI::line() followed by an exit().

​https://github.com/wp-cli/wp-cli/issues/2386 was fixed in v0.23.0.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/inc/cli/class-set-plugin-project.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-plugin-directory/inc/cli/class-set-plugin-project.php

@@ -37 +37 @@
     public function __invoke( $args, $assoc_args ) {
         if ( ! preg_match( '/^[^\/]+$/', $args[0] ) ) {
-            WP_CLI::line( sprintf( "The plugin slug '%s' contains invalid characters.", $args[0] ) );
-            exit( 1 );
+            WP_CLI::error( sprintf( "The plugin slug '%s' contains invalid characters.", $args[0] ), 1 );
         }
 
@@ -48 +47 @@
         $master_project = GP::$project->by_path( $this->master_project_path );
         if ( ! $master_project ) {
-            WP_CLI::line( sprintf( "The master project '%s' couldn't be found.", $this->master_project_path ) );
-            exit( 2 );
+            WP_CLI::error( sprintf( "The master project '%s' couldn't be found.", $this->master_project_path ), 2 );
         }
 
@@ -86 +84 @@
             $plugin_project = GP::$project->create_and_select( $project_args );
             if ( ! $plugin_project ) {
-                WP_CLI::line( "Couldn't create project for {$plugin_project_path}." );
-                exit( 5 );
+                WP_CLI::error( "Couldn't create project for {$plugin_project_path}.", 5 );
             }
         } else {