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

Timestamp:

11/23/2017 04:08:42 AM (7 years ago)

Author:

pento

Message:

General: Reformat inline if () statements inside HTML tags.

This pattern occurs a handful of times across the codebase:

<div class="foo<?php if ( $bar ) { echo ' baz'; } ?>">

Unfortunately, it doesn't really play nicely with phpcbf, so all instances need to be removed in preperation for auto code formatting.

See #41057.

File:

• trunk/src/wp-admin/themes.php (modified) (1 diff)

trunk/src/wp-admin/themes.php

@@ -251 +251 @@
     $aria_action = esc_attr( $theme['id'] . '-action' );
     $aria_name   = esc_attr( $theme['id'] . '-name' );
+
+    $active_class = '';
+    if ( $theme['active'] ) {
+        $active_class = ' active';
+    }
     ?>
-<div class="theme<?php if ( $theme['active'] ) echo ' active'; ?>" tabindex="0" aria-describedby="<?php echo $aria_action . ' ' . $aria_name; ?>">
+<div class="theme<?php echo $active_class; ?>" tabindex="0" aria-describedby="<?php echo $aria_action . ' ' . $aria_name; ?>">
     <?php if ( ! empty( $theme['screenshot'][0] ) ) { ?>
         <div class="theme-screenshot">