Changeset 29924

Timestamp:

10/16/2014 07:45:26 PM (11 years ago)

Author:

ocean90

Message:

Themes: Make it possible to delete broken themes.

props ideag, obenland.
fixes #28165.

Location:

trunk/src/wp-admin

Files:

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

trunk/src/wp-admin/css/themes.css

@@ -581 +581 @@
 }
 
+.broken-themes a.delete-theme,
 .theme-overlay .theme-actions .delete-theme {
     color: #a00;
-    position: absolute;
-    right: 10px;
-    bottom: 5px;
     text-decoration: none;
     border-color: transparent;
@@ -593 +591 @@
 }
 
+.theme-overlay .theme-actions .delete-theme {
+    position: absolute;
+    right: 10px;
+    bottom: 5px;
+}
+
+.broken-themes a.delete-theme:hover,
+.broken-themes a.delete-theme:focus,
 .theme-overlay .theme-actions .delete-theme:hover,
 .theme-overlay .theme-actions .delete-theme:focus {
@@ -1730 +1736 @@
         padding: 0;
     }
-}
+
+    .broken-themes table {
+        width: 100%;
+    }
+}

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

@@ -1660 +1660 @@
         themes.Run.init();
     }
+
+    $( '.broken-themes .delete-theme' ).on( 'click', function() {
+        return confirm( _wpThemeSettings.settings.confirmDelete );
+    });
 });
 

trunk/src/wp-admin/themes.php

@@ -247 +247 @@
 <p><?php _e('The following themes are installed but incomplete. Themes must have a stylesheet and a template.'); ?></p>
 
+<?php
+$can_delete = current_user_can( 'delete_themes' );
+?>
 <table>
     <tr>
         <th><?php _ex('Name', 'theme name'); ?></th>
         <th><?php _e('Description'); ?></th>
+        <?php if ( $can_delete ) { ?>
+            <th></th>
+        <?php } ?>
+        </tr>
     </tr>
-<?php
-    foreach ( $broken_themes as $broken_theme ) {
-        echo "
+    <?php foreach ( $broken_themes as $broken_theme ) : ?>
         <tr>
-             <td>" . ( $broken_theme->get( 'Name' ) ? $broken_theme->get( 'Name' ) : $broken_theme->get_stylesheet() ) . "</td>
-             <td>" . $broken_theme->errors()->get_error_message() . "</td>
-        </tr>";
-    }
-?>
+            <td><?php echo $broken_theme->get( 'Name' ) ? $broken_theme->display( 'Name' ) : $broken_theme->get_stylesheet(); ?></td>
+            <td><?php echo $broken_theme->errors()->get_error_message(); ?></td>
+            <?php
+            if ( $can_delete ) {
+                $stylesheet = $broken_theme->get_stylesheet();
+                $delete_url = add_query_arg( array(
+                    'action'     => 'delete',
+                    'stylesheet' => urlencode( $stylesheet ),
+                ), admin_url( 'themes.php' ) );
+                $delete_url = wp_nonce_url( $delete_url, 'delete-theme_' . $stylesheet );
+                ?>
+                <td><a href="<?php echo esc_url( $delete_url ); ?>" class="button button-secondary delete-theme"><?php _e( 'Delete' ); ?></a></td>
+                <?php
+            }
+            ?>
+        </tr>
+    <?php endforeach; ?>
 </table>
 </div>