Changeset 13636 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

Timestamp:

05/01/2024 01:23:04 AM (11 months ago)

Author:

dd32

Message:

Plugin Directory: Review: Suffix ZIP urls with the API through .zip#wporgapi:https://.... rather than through .zip?info=https...

This will improve the experience for reviewers, as otherwise ? needs to be escaped on the command line.

See #7385.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php (modified) (1 diff)

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

@@ -1853 +1853 @@
         }
 
-        $url = add_query_arg(
-            'info',
-            urlencode( rest_url( sprintf(
-                'plugins/v1/pending-plugin/%d-%s/',
-                $post->ID,
-                $token
-            ) ) ),
-            $url
-        );
+        // Append with a anchor, such that CLI environments don't require special handling.
+        $url .= '#wporgapi:' . rest_url( sprintf(
+            'plugins/v1/pending-plugin/%d-%s/',
+            $post->ID,
+            $token
+        ) );
 
         return $url;