Changeset 13512

Timestamp:

04/12/2024 05:24:54 AM (9 months ago)

Author:

dd32

Message:

Plugin Directory: Add missed changes in [13507].

See #7386.

Location:

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

Files:

• admin/class-customizations.php (modified) (1 diff)
• js/edit-form.js (modified) (1 diff)

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

@@ -551 +551 @@
         $plugin = get_post( $postarr['ID'] );
         if ( $plugin && $plugin->post_name !== $data['post_name'] ) {
-            Tools::audit_log( sprintf(
-                "Slug changed from '%s' to '%s'.",
-                $plugin->post_name,
-                $data['post_name']
-            ), $plugin->ID );
+            // Only log if the slugs don't appear to be rejection-related.
+            if (
+                ! preg_match( '!^rejected-.+-rejected$!', $post->post_name ) &&
+                ! preg_match( '!^rejected-.+-rejected$!', $data['post_name'] )
+            ) {
+                Tools::audit_log( sprintf(
+                    "Slug changed from '%s' to '%s'.",
+                    $plugin->post_name,
+                    $data['post_name']
+                ), $plugin->ID );
+            }
         }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/js/edit-form.js

@@ -53 +53 @@
 
         setPluginStatus: function() {
-            if ( 'approved' === $(this).val() ) {
+            var $this = $(this),
+                status = $this.val();
+
+            if ( 'new' == status ) {
+                jQuery('#assigned_reviewer').val(0);
+
+            } else if ( 'pending' == status && $this.hasClass('pending-and-assign') ) {
+                jQuery('#assigned_reviewer').val( userSettings.uid );
+
+            } else if ( 'approved' === status ) {
                 return confirm( pluginDirectory.approvePluginAYS );
-            } else if ( 'rejected' === $(this).val() ) {
+
+            } else if ( 'rejected' === status ) {
                 return confirm( pluginDirectory.rejectPluginAYS );
+
             } else {
                 return true;