Changeset 24123 for trunk/wp-admin/includes/class-wp-terms-list-table.php

Timestamp:

04/29/2013 01:10:50 AM (11 years ago)

Author:

SergeyBiryukov

Message:

Remove redundant echo calls from list tables. Don't mix string concatenation with direct output. see #24210.

File:

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

trunk/wp-admin/includes/class-wp-terms-list-table.php

@@ -137 +137 @@
 
         // convert it to table rows
-        $out = '';
         $count = 0;
 
@@ -153 +152 @@
 
             // Some funky recursion to get the job done( Paging & parents mainly ) is contained within, Skip it for non-hierarchical taxonomies for performance sake
-            $out .= $this->_rows( $taxonomy, $terms, $children, $offset, $number, $count );
+            $this->_rows( $taxonomy, $terms, $children, $offset, $number, $count );
         } else {
             $terms = get_terms( $taxonomy, $args );
             foreach ( $terms as $term )
-                $out .= $this->single_row( $term, 0, $taxonomy );
+                $this->single_row( $term, 0, $taxonomy );
             $count = $number; // Only displaying a single page.
         }
@@ -166 +165 @@
             $this->no_items();
             echo '</td></tr>';
-        } else {
-            echo $out;
         }
     }
@@ -175 +172 @@
         $end = $start + $per_page;
 
-        $output = '';
         foreach ( $terms as $key => $term ) {
 
@@ -200 +196 @@
                 $num_parents = count( $my_parents );
                 while ( $my_parent = array_pop( $my_parents ) ) {
-                    $output .=  "\t" . $this->single_row( $my_parent, $level - $num_parents, $taxonomy );
+                    echo "\t";
+                    $this->single_row( $my_parent, $level - $num_parents, $taxonomy );
                     $num_parents--;
                 }
             }
 
-            if ( $count >= $start )
-                $output .= "\t" . $this->single_row( $term, $level, $taxonomy );
+            if ( $count >= $start ) {
+                echo "\t";
+                $this->single_row( $term, $level, $taxonomy );
+            }
 
             ++$count;
@@ -213 +212 @@
 
             if ( isset( $children[$term->term_id] ) && empty( $_REQUEST['s'] ) )
-                $output .= $this->_rows( $taxonomy, $terms, $children, $start, $per_page, $count, $term->term_id, $level + 1 );
-        }
-
-        return $output;
+                $this->_rows( $taxonomy, $terms, $children, $start, $per_page, $count, $term->term_id, $level + 1 );
+        }
     }
 
@@ -226 +223 @@
 
         echo '<tr id="tag-' . $tag->term_id . '"' . $row_class . '>';
-        echo $this->single_row_columns( $tag );
+        $this->single_row_columns( $tag );
         echo '</tr>';
     }