Changeset 26288

Timestamp:

11/20/2013 02:08:17 PM (12 years ago)

Author:

SergeyBiryukov

Message:

Only show "Add New Theme" block on Manage Themes screen if not Multisite and the user can install themes.

props markoheijnen.
see #25948.

Location:

trunk/src/wp-admin

Files:

• js/theme.js (modified) (1 diff)
• themes.php (modified) (2 diffs)

trunk/src/wp-admin/js/theme.js

@@ -466 +466 @@
 
         // 'Add new theme' element shown at the end of the grid
-        this.$el.append( '<div id="add-new" class="theme add-new"><a href="' + themes.data.settings.install_uri + '"><div class="theme-screenshot"><span></span></div><h3 class="theme-name">' + themes.data.i18n.add_new + '</h3></a></div>' );
+        if ( themes.data.settings.canInstall ) {
+            this.$el.append( '<div id="add-new" class="theme add-new"><a href="' + themes.data.settings.installURI + '"><div class="theme-screenshot"><span></span></div><h3 class="theme-name">' + themes.data.i18n.addNew + '</h3></a></div>' );
+        }
 
         this.parent.page++;

trunk/src/wp-admin/themes.php

@@ -92 +92 @@
     'themes'   => $themes,
     'settings' => array(
-        'install_uri'   => admin_url( 'theme-install.php' ),
+        'canInstall'    => ( ! is_multisite() && current_user_can( 'install_themes' ) ),
+        'installURI'    => admin_url( 'theme-install.php' ),
         'customizeURI'  => ( current_user_can( 'edit_theme_options' ) ) ? wp_customize_url() : null,
         'confirmDelete' => __( "Are you sure you want to delete this theme?\n\nClick 'Cancel' to go back, 'OK' to confirm the delete." ),
@@ -100 +101 @@
     ),
     'i18n' => array(
-        'add_new'         => __( 'Add New Theme' ),
+        'addNew'        => __( 'Add New Theme' ),
     ),
 ) );