Changeset 5210

Timestamp:

03/30/2017 03:57:21 PM (8 years ago)

Author:

obenland

Message:

Plugin Directory: Move admin link into meta widget

It's pretty meta after all.

See #2111.

Location:

sites/trunk/wordpress.org/public_html/wp-content

Files:

• plugins/plugin-directory/widgets/class-meta.php (modified) (2 diffs)
• themes/pub/wporg-plugins/template-parts/plugin-single.php (modified) (1 diff)

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

@@ -46 +46 @@
                 <li><?php printf( __( 'Designed to work with: %s', 'wporg-plugins' ), $built_for ); ?></li>
             <?php endif; ?>
+
             <li><?php printf( __( 'Version: %s', 'wporg-plugins' ), '<strong>' . get_post_meta( $post->ID, 'version', true ) . '</strong>' ); ?></li>
             <li>
@@ -56 +57 @@
             </li>
             <li><?php printf( __( 'Active installs: %s', 'wporg-plugins' ), '<strong>' . Template::active_installs( false ) . '</strong>' ); ?></li>
-            <?php if ( $tested_up_to = (string) get_post_meta( $post->ID, 'tested', true ) ) { ?>
+
+            <?php if ( $tested_up_to = (string) get_post_meta( $post->ID, 'tested', true ) ) : ?>
                 <li><?php printf( __( 'Tested up to: %s', 'wporg-plugins' ), '<strong>' . $tested_up_to . '</strong>' ); ?></li>
-            <?php } ?>
+            <?php endif; ?>
+
             <?php if ( $tags = get_the_term_list( $post->ID, 'plugin_tags', '<div class="tags">', '', '</div>' ) ) : ?>
-                <li><?php
+                <li class="clear"><?php
                     $terms = get_the_terms( $post, 'plugin_tags' );
-                    if ( 1 == count( $terms ) ) {
+                    if ( 1 == count( $terms ) ) :
                         printf( __( 'Tag: %s', 'wporg-plugins' ), $tags );
-                    } else {
+                    else :
                         printf( __( 'Tags: %s', 'wporg-plugins' ), $tags );
-                    }
+                    endif;
                 ?></li>
+            <?php endif; ?>
+
+            <?php if ( current_user_can( 'plugin_admin_view', $post ) ) : ?>
+                <li class="hide-if-no-js">
+                    <?php
+                        printf( __( 'Admin access: %s', 'wporg-plugins' ),
+                            sprintf( '<strong><a class="plugin-admin" href="%s">%s</a></strong>', esc_url( get_permalink() . 'admin/' ), __( 'View Admin', 'wporg-plugins' ) )
+                        );
+                    ?>
+                </li>
             <?php endif; ?>
         </ul>

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/template-parts/plugin-single.php

@@ -95 +95 @@
                 </div>
             <?php endif; ?>
-            <div>
-                <a class="plugin-download button download-button button-large" href="<?php echo esc_url( Template::download_link() ); ?>" itemprop="downloadUrl"><?php _e( 'Download', 'wporg-plugins' ); ?></a>
-                <?php if ( current_user_can( 'plugin_admin_view', get_post() ) ) : ?>
-                    <br>
-                    <a class="plugin-edit hide-if-no-js" href="<?php echo esc_url( get_permalink() . 'admin/' ); ?>"><?php _e( 'Admin', 'wporg-plugins' ); ?></a>
-                <?php endif; ?>
-            </div>
 
+            <a class="plugin-download button download-button button-large" href="<?php echo esc_url( Template::download_link() ); ?>" itemprop="downloadUrl"><?php _e( 'Download', 'wporg-plugins' ); ?></a>
             <meta itemprop="softwareVersion" content="<?php echo esc_attr( get_post_meta( get_the_ID(), 'version', true ) ); ?>">
             <meta itemprop="fileFormat" content="application/zip">