Ticket #2800: 2800.2.diff

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

@@ -36 +36 @@
                 if ( current_user_can( 'plugin_add_committer', $post ) || current_user_can( 'plugin_remove_committer', $post ) ) {
                         wp_enqueue_script( 'wporg-plugins-committers', plugins_url( 'js/committers.js', __FILE__ ), array( 'wp-util' ), true );
                         wp_localize_script( 'wporg-plugins-committers', 'committersWidget', array(
-                                'restUrl'    => get_rest_url(),
-                                'restNonce'  => wp_create_nonce( 'wp_rest' ),
-                                'pluginSlug' => $post->post_name,
+                                'restUrl'            => get_rest_url(),
+                                'restNonce'          => wp_create_nonce( 'wp_rest' ),
+                                'pluginSlug'         => $post->post_name,
+                                'removeCommitterAYS' => __( 'Are you sure you want to remove this committer?', 'wporg-plugins' ),
                         ) );
                 }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/widgets/js/committers.js

@@ -18 +18 @@
 
                 $( '#committer-list' )
                         .on( 'click', '.remove', function() {
+                                if ( ! window.confirm( pluginDir.removeCommitterAYS ) ) {
+                                        return;
+                                }
+
                                 var $row = $( this ).addClass( 'spinner' ).parents( 'li' ),
                                         url = pluginDir.restUrl + 'plugins/v1/plugin/' + pluginDir.pluginSlug + '/committers/' + $row.data( 'user' ) + '/?_wpnonce=' + pluginDir.restNonce;