Changeset 2978

Timestamp:

04/19/2016 07:57:19 PM (9 years ago)

Author:

obenland

Message:

Plugins Directory: Always show link to zip file if available.

Restricting it to only pending does not reflect the review flow anymore.

See #1570.

File:

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

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

@@ -9 +9 @@
 class Review_Tools {
     static function display() {
-        $post = get_post();
-        $plugin_slug = $post->post_name;
+        $post      = get_post();
+        $zip_files = get_attached_media( 'application/zip', $post );
 
-        $zip_files = get_attached_media( 'application/zip', $post );
-        if ( $zip_files && 'pending' == $post->post_status ) {
+        if ( $zip_files ) {
             $zip_file = current( $zip_files );
-            $zip_url = wp_get_attachment_url( $zip_file->ID );
-            printf(
-                '<p>' . __( '<strong>Zip file:</strong> %s', 'wporg-plugins' ) . '</p>',
-                sprintf(
-                    '<a href="%s">%s</a>',
-                    esc_url( $zip_url ),
-                    esc_html( $zip_url )
-                )
+            $zip_url  = wp_get_attachment_url( $zip_file->ID );
+            printf( '<p>' . __( '<strong>Zip file:</strong> %s', 'wporg-plugins' ) . '</p>',
+                sprintf( '<a href="%s">%s</a>', esc_url( $zip_url ), esc_html( $zip_url ) )
             );
         }
 
         echo "<ul>
-            <li><a href='https://plugins.trac.wordpress.org/log/{$plugin_slug}/'>" . __( 'Development Log', 'wporg-plugins' ) . "</a></li>
-            <li><a href='https://plugins.svn.wordpress.org/{$plugin_slug}/'>" . __( 'Subversion Repository', 'wporg-plugins' ) . "</a></li>
-            <li><a href='https://plugins.trac.wordpress.org/browser/{$plugin_slug}/'>" . __( 'Browse in Trac', 'wporg-plugins' ) . '</a></li>
+            <li><a href='https://plugins.trac.wordpress.org/log/{$post->post_name}/'>" . __( 'Development Log', 'wporg-plugins' ) . "</a></li>
+            <li><a href='https://plugins.svn.wordpress.org/{$post->post_name}/'>" . __( 'Subversion Repository', 'wporg-plugins' ) . "</a></li>
+            <li><a href='https://plugins.trac.wordpress.org/browser/{$post->post_name}/'>" . __( 'Browse in Trac', 'wporg-plugins' ) . '</a></li>
         </ul>';