Changeset 16545

Timestamp:

11/23/2010 11:39:50 AM (13 years ago)

Author:

westi

Message:

Implement proper column show/hide behaviour based on screen options for the Network Admin Sites table.

File:

• trunk/wp-admin/includes/class-wp-ms-sites-list-table.php (modified) (7 diffs)

trunk/wp-admin/includes/class-wp-ms-sites-list-table.php

@@ -201 +201 @@
 
             foreach ( $columns as $column_name => $column_display_name ) {
+                $style = '';
+                if ( in_array( $column_name, $hidden ) )
+                    $style = ' style="display:none;"';
+
                 switch ( $column_name ) {
                     case 'cb': ?>
@@ -209 +213 @@
                     break;
 
-                    case 'id': ?>
+                    case 'id':?>
                         <th valign="top" scope="row">
                             <?php echo $blog['blog_id'] ?>
@@ -216 +220 @@
                     break;
 
-                    case 'blogname': ?>
-                        <td class="column-title">
+                    case 'blogname':
+                        echo "<td class='column-$column_name $column_name'$style>"; ?>
                             <a href="<?php echo esc_url( network_admin_url( 'site-info.php?id=' . $blog['blog_id'] ) ); ?>" class="edit"><?php echo $blogname . $blog_state; ?></a>
                             <?php
@@ -263 +267 @@
                     break;
 
-                    case 'lastupdated': ?>
-                        <td valign="top">
-                            <?php
+                    case 'lastupdated':
+                        echo "<td valign='top'class='$column_name column-$column_name'$style>";
                             if ( 'list' == $mode )
                                 $date = 'Y/m/d';
@@ -274 +277 @@
                     <?php
                     break;
-                case 'registered': ?>
-                        <td valign="top">
-                        <?php
+                case 'registered':
+                        echo "<td valign='top'class='$column_name column-$column_name'$style>";
                         if ( $blog['registered'] == '0000-00-00 00:00:00' )
                             echo '&#x2014;';
@@ -285 +287 @@
                     <?php
                     break;
-                case 'users': ?>
-                        <td valign="top">
-                            <?php
+                case 'users':
+                        echo "<td valign='top'class='$column_name column-$column_name'$style>";
                             $blogusers = get_users( array( 'blog_id' => $blog['blog_id'], 'number' => 6) );
                             if ( is_array( $blogusers ) ) {
@@ -317 +318 @@
                     <?php break;
 
-                default: ?>
-                        <?php if ( has_filter( 'manage_blogs_custom_column' ) ) { ?>
-                        <td valign="top">
-                            <?php do_action( 'manage_blogs_custom_column', $column_name, $blog['blog_id'] ); ?>
-                        </td>
-                        <?php } ?>
-                    <?php break;
+                default:
+                    echo "<td class='$column_name column-$column_name'$style>";
+                    do_action( 'manage_themes_custom_column', $column_name, $theme_key, $theme );
+                    echo "</td>";
+                    break;
                 }
             }