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

Timestamp:

02/04/2018 11:26:05 AM (9 years ago)

Author:

ocean90

Message:

Translate, Plugin Directory: Fill plugin details for existing projects with old data in case the API is unreachable.

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

@@ -45 +45 @@
                 }
 
+                $plugin_project = GP::$project->by_path( $plugin_project_path );
+
                 // Get plugin details from the API.
                 $plugin_details = $this->get_plugin_details( $plugin_slug );
@@ -50 +52 @@
                         WP_CLI::warning( "The plugin API couldn't be reached." );
 
-                        /*
-                         * That's usally the case for the initial commit.
-                         * Provide a mockup which contains the necessary plugin details.
-                         */
-                        $plugin_details = new stdClass();
-                        $plugin_details->name = $plugin_slug;
-                        $plugin_details->short_description = '';
-                        $plugin_details->stable_tag = 'trunk';
+                        if ( ! $plugin_project ) {
+                                /*
+                                 * That's usally the case for the initial commit.
+                                 * Provide a mockup which contains the necessary plugin details.
+                                 */
+                                $plugin_details = new stdClass();
+                                $plugin_details->name = $plugin_slug;
+                                $plugin_details->short_description = '';
+                                $plugin_details->stable_tag = 'trunk';
+                        } else {
+                                // Fill details with old data in case the API is down.
+                                $plugin_details = new stdClass();
+                                $plugin_details->name = $plugin_project->name;
+                                $plugin_details->short_description = $plugin_project->description;
+                                $plugin_details->stable_tag = 'trunk';
+                        }
                 }
 
@@ -68 +78 @@
                         'active'            => 1,
                 );
-
-                $plugin_project = GP::$project->by_path( $plugin_project_path );
 
                 // Is this a new or an existing project?