Changeset 13440 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php

Timestamp:

04/03/2024 05:46:47 AM (13 months ago)

Author:

dd32

Message:

Plugin Directory: Split the SVN repo creation step out from the approval function, to allow manually calling when it fails.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-status-transitions.php

@@ -211 +211 @@
 
         // Create SVN repo.
+        $this->approved_create_svn_repo( $post, $plugin_author );
+
+        // Grant commit access.
+        Tools::grant_plugin_committer( $post->post_name, $plugin_author );
+
+        // Send email.
+        $email = new Plugin_Approved_Email( $post, $plugin_author );
+        $email->send();
+
+        Tools::audit_log( 'Plugin approved.', $post_id );
+    }
+
+    /**
+     * Create a SVN repository for this plugin.
+     *
+     * @param \WP_Post $post          Post object.
+     * @param \WP_User $plugin_author Plugin author.
+     * @return bool
+     */
+    public function approved_create_svn_repo( $post, $plugin_author ) {
         $dir = Filesystem::temp_directory( $post->post_name );
         foreach ( array( 'assets', 'tags', 'trunk' ) as $folder ) {
@@ -232 +252 @@
         */
 
-        SVN::import( $dir, 'http://plugins.svn.wordpress.org/' . $post->post_name, sprintf( 'Adding %1$s by %2$s.', $post->post_title, $plugin_author->user_login ) );
-
-        // Grant commit access.
-        Tools::grant_plugin_committer( $post->post_name, $plugin_author );
-
-        // Send email.
-        $email = new Plugin_Approved_Email( $post, $plugin_author );
-        $email->send();
-
-        Tools::audit_log( 'Plugin approved.', $post_id );
+        $result = SVN::import( $dir, 'http://plugins.svn.wordpress.org/' . $post->post_name, sprintf( 'Adding %1$s by %2$s.', $post->post_title, $plugin_author->user_login ) );
+
+        if ( $result['errors'] ) {
+            Tools::audit_log( 'Error creating SVN repository: ' . var_export( $result['errors'], true ), $post->ID );
+            return false;
+        }
+
+        return true;
     }