Changeset 2753 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-committers.php

Timestamp:

03/15/2016 09:11:27 PM (9 years ago)

Author:

obenland

Message:

Plugin Directory: After [2752], improve error handling and docs.

See #1571.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-committers.php (modified) (6 diffs)

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

@@ -22 +22 @@
     }
 
+    /**
+     * Displays a list of committers for the current plugin.
+     */
     public static function display() {
         $list = new Committers_List_Table();
@@ -28 +31 @@
     }
 
+    /**
+     * Ajax handler for adding a new committer.
+     */
     public static function add_committer() {
         check_ajax_referer( 'add-committer' );
 
-        $login   = isset( $_POST['add_committer'] ) ? sanitize_user( $_POST['add_committer'] ) : '';
-        $post_id = isset( $_POST['post_id'] ) ? (int) $_POST['post_id'] : 0;
+        $login    = isset( $_POST['add_committer'] ) ? sanitize_user( $_POST['add_committer'] ) : '';
+        $post_id  = isset( $_POST['post_id'] ) ? (int) $_POST['post_id'] : 0;
+        $response = new \WP_Ajax_Response();
 
         if ( ! $committer = get_user_by( 'login', $login ) ) {
-            wp_die( time() );
+            $response->add( array(
+                'what' => 'committer',
+                'data' => new \WP_Error( 'error', sprintf( __( 'The user %s does not exist.', 'wporg-plugins' ), '<code>' . $login . '</code>' ) ),
+            ) );
+            $response->send();
         }
 
-        if ( ! current_user_can( 'manage_committers', $post_id ) ) {
-        //  wp_die( -1 );
+        // @todo: Capabilities.
+        if ( ! current_user_can( 'add_committers', $post_id ) ) {
+            //  wp_die( -1 );
         }
         global $post, $wpdb;
 
-        $response = new \WP_Ajax_Response();
-        $post     = get_post( $post_id );
-
+        $post   = get_post( $post_id );
         $result = $wpdb->insert( PLUGINS_TABLE_PREFIX . 'svn_access', array(
             'path'   => "/{$post->post_name}",
@@ -51 +61 @@
             'access' => 'rw',
         ) );
-
         if ( ! $result ) {
-            $message = __( 'An error has occurred. Please reload the page and try again.' );
-            if ( is_wp_error( $result ) && $result->get_error_message() ) {
-                $message = $result->get_error_message();
+            if ( 'Duplicate entry' === substr( $wpdb->last_error, 0, 15 ) ) {
+                $message = __( 'Duplicate committer detected.', 'wporg-plugins' );
+            } else {
+                $message = __( 'An error has occurred. Please reload the page and try again.', 'wporg-plugins' );
             }
 
@@ -76 +86 @@
     }
 
+    /**
+     * Ajax handler for removing a committer.
+     */
     public static function remove_committer() {
         $id      = isset( $_POST['id'] )      ? (int) $_POST['id']      : 0;
@@ -82 +95 @@
         check_ajax_referer( "remove-committer-$id" );
 
+        $response = new \WP_Ajax_Response();
+
         if ( ! $committer = get_user_by( 'id', $id ) ) {
-            wp_die( time() );
+            $response->add( array(
+                'what' => 'committer',
+                'data' => new \WP_Error( 'error', sprintf( __( 'The user %s does not exist.', 'wporg-plugins' ), '<code>' . $login . '</code>' ) ),
+            ) );
+            $response->send();
         }
 
-        if ( ! current_user_can( 'manage_committers', $post_id ) ) {
-        //  wp_die( -1 );
+        // @todo: Capabilities.
+        if ( ! current_user_can( 'remove_committers', $post_id ) ) {
+            //  wp_die( -1 );
         }
 
@@ -93 +113 @@
 
         $result = $GLOBALS['wpdb']->delete( PLUGINS_TABLE_PREFIX . 'svn_access', array(
-            'path'   => "/{$plugin_slug}",
-            'user'   => $committer->user_login,
+            'path' => "/{$plugin_slug}",
+            'user' => $committer->user_login,
         ) );