Changeset 25666 for trunk/src/wp-admin/themes.php

Timestamp:

10/02/2013 08:49:59 PM (12 years ago)

Author:

nacin

Message:

Show a proper error message on the dashboard when the current theme is broken.

Clean up Appearance -> Themes when we don't have enough information to show for a broken theme.

Clean up broken/ugly "alt" row styling when we are displaying broken themes.

props johnbillion for initial patch.
fixes #21670.

File:

• trunk/src/wp-admin/themes.php (modified) (3 diffs)

trunk/src/wp-admin/themes.php

@@ -139 +139 @@
         <?php echo $ct->display('Name'); ?>
     </h4>
+
+<?php
+if ( $ct->errors() && ( ! is_multisite() || current_user_can( 'manage_network_themes' ) ) ) {
+    echo '<p class="error-message">' . sprintf( __( 'ERROR: %s' ), $ct->errors()->get_error_message() ) . '</p>';
+}
+
+// Certain error codes are less fatal than others. We can still display theme information in most cases.
+if ( ! $ct->errors() || ( 1 == count( $ct->errors()->get_error_codes() )
+    && in_array( $ct->errors()->get_error_code(), array( 'theme_no_parent', 'theme_parent_invalid', 'theme_no_index' ) ) ) ) : ?>
 
     <div>
@@ -208 +217 @@
     ?>
 
+<?php endif; // theme errors ?>
+
 </div>
 
@@ -293 +304 @@
     </tr>
 <?php
-    $alt = '';
     foreach ( $broken_themes as $broken_theme ) {
-        $alt = ('class="alternate"' == $alt) ? '' : 'class="alternate"';
         echo "
-        <tr $alt>
+        <tr>
              <td>" . $broken_theme->get('Name') ."</td>
              <td>" . $broken_theme->errors()->get_error_message() . "</td>