Changeset 10692

Timestamp:

02/17/2021 10:13:23 PM (5 years ago)

Author:

coffee2code

Message:

Developer, CLI: Automatically install latest WP for parsing into temp directory instead of requiring it be done manually beforehand.

Supports parsing of an explicit path via optional --src_path argument.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/cli-commands.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-developer/inc/cli-commands.php

@@ -14 +14 @@
          * ## OPTIONS
          *
-         * <path>
+         * [--src_path=<src_path>]
          * : The path to a copy of WordPress to be parsed. Should not be code used in
-         * an active install.
+         * an active install. If not defined, then the latest version of WordPress will
+         * be downloaded to a temp directory and parsed.
          *
          * [--user_id=<user_id>]
@@ -23 +24 @@
          * ## EXAMPLES
          *
-         *     # Parse WP 4.9.5 into DevHub.
-         *     $ wp devhub /home/wporgdev/developer/wp495
+         *     # Parse lastest WP.
+         *     $ wp devhub parse
+         *
+         *     # Parse specific copy of WP.
+         *     $ wp devhub parse --src_path=/path/to/wordpress
          *
          * @when after_wp_load
          */
         public function parse( $args, $assoc_args ) {
-                list( $path ) = $args;
-
+                $path = $assoc_args['src_path'] ?? null;
+
+                // Verify path is a file or directory.
+                if ( $path ) {
+                        if ( file_exists( $path ) ) {
+                                WP_CLI::log( 'Parsing WordPress source from specified directory: ' . $path );
+                        } else {
+                                WP_CLI::error( 'Provided path for WordPress source to parse does not exist.' );
+                        }
+                 }
+ 
+                // If no path provided, use a temporary path.
+                if ( ! $path ) {
+                        $path = WP_CLI\Utils\get_temp_dir() . 'devhub_' . time();
+         
+                        // @todo Attempt to reuse an existing temp dir.
+                        if ( mkdir( $path ) ) {
+                                WP_CLI::log( "Installing latest WordPress into temporary directory ({$path})..." );
+                                $cmd = "core download --path={$path}";
+                                // Install WP into the temp directory.
+                                WP_CLI::runcommand( $cmd, [] );
+                        } else {
+                                $path = null;
+                        }
+                }
+         
+                if ( ! $path ) {
+                        WP_CLI::error( 'Unable to create temporary directory for downloading WordPress. If retrying fails, consider obtaining the files manually and supplying that path via --src_path argument.' );
+                }
+
+                // Determine importing user's ID. 
                 $user_id = $assoc_args['user_id'] ?? 5911429; // 5911429 = ID for wordpressdotorg
                 $user = get_user_by( 'id', $user_id );
@@ -42 +75 @@
                         'posts-to-posts' => 'posts-to-posts/posts-to-posts.php',
                 ];
-
-                // Verify path is a file or directory.
-                if ( ! file_exists( $path ) ) {
-                        WP_CLI::error( 'Path for WordPress source to parse does not exist.' );
-                }
 
                 // Verify path is not a file.