Ticket #2800: 2800.1.diff

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,
+                                'confirmDeleteComitter' => __( 'Are you sure you want to remove commit access for the user', 'wporg-plugins' )
                         ) );
                 }
 

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

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