Changeset 28405

Timestamp:

05/15/2014 02:17:52 AM (10 years ago)

Author:

wonderboymusic

Message:

Eliminate use of extract() in wp_list_bookmarks().

See #22400.

File:

• trunk/src/wp-includes/bookmark-template.php (modified) (6 diffs)

trunk/src/wp-includes/bookmark-template.php

@@ -139 +139 @@
             );
         }
-        $output .= $r['$after'] . "\n";
+        $output .= $r['after'] . "\n";
     } // end while
 
@@ -211 +211 @@
  *      Default is not return anything.
  */
-function wp_list_bookmarks($args = '') {
+function wp_list_bookmarks( $args = '' ) {
     $defaults = array(
         'orderby' => 'name', 'order' => 'ASC',
@@ -225 +225 @@
 
     $r = wp_parse_args( $args, $defaults );
-    extract( $r, EXTR_SKIP );
 
     $output = '';
 
-    if ( $categorize ) {
-        $cats = get_terms( 'link_category', array( 'name__like' => $category_name, 'include' => $category, 'exclude' => $exclude_category, 'orderby' => $category_orderby, 'order' => $category_order, 'hierarchical' => 0 ) );
-        if ( empty( $cats ) )
+    if ( $r['categorize'] ) {
+        $cats = get_terms( 'link_category', array(
+            'name__like' => $r['category_name'],
+            'include' => $r['category'],
+            'exclude' => $r['exclude_category'],
+            'orderby' => $r['category_orderby'],
+            'order' => $r['category_order'],
+            'hierarchical' => 0
+        ) );
+        if ( empty( $cats ) ) {
             $categorize = false;
+        }
     }
 
@@ -238 +245 @@
         // Split the bookmarks into ul's for each category
         foreach ( (array) $cats as $cat ) {
-            $params = array_merge($r, array('category'=>$cat->term_id));
-            $bookmarks = get_bookmarks($params);
-            if ( empty($bookmarks) )
+            $params = array_merge( $r, array( 'category' => $cat->term_id ) );
+            $bookmarks = get_bookmarks( $params );
+            if ( empty( $bookmarks ) ) {
                 continue;
-            $output .= str_replace(array('%id', '%class'), array("linkcat-$cat->term_id", $class), $category_before);
+            }
+            $output .= str_replace(
+                array( '%id', '%class' ),
+                array( "linkcat-$cat->term_id", $r['class'] ),
+                $r['category_before']
+            );
             /**
              * Filter the bookmarks category name.
@@ -252 +264 @@
             $catname = apply_filters( 'link_category', $cat->name );
 
-            $output .= "$title_before$catname$title_after\n\t<ul class='xoxo blogroll'>\n";
-            $output .= _walk_bookmarks($bookmarks, $r);
-            $output .= "\n\t</ul>\n$category_after\n";
+            $output .= $r['title_before'];
+            $output .= $catname;
+            $output .= $r['title_after'];
+            $output .= "\n\t<ul class='xoxo blogroll'>\n";
+            $output .= _walk_bookmarks( $bookmarks, $r );
+            $output .= "\n\t</ul>\n";
+            $output .= $r['category_after'] . "\n";
         }
     } else {
         //output one single list using title_li for the title
-        $bookmarks = get_bookmarks($r);
-
-        if ( !empty($bookmarks) ) {
-            if ( !empty( $title_li ) ){
-                $output .= str_replace(array('%id', '%class'), array("linkcat-$category", $class), $category_before);
-                $output .= "$title_before$title_li$title_after\n\t<ul class='xoxo blogroll'>\n";
-                $output .= _walk_bookmarks($bookmarks, $r);
-                $output .= "\n\t</ul>\n$category_after\n";
+        $bookmarks = get_bookmarks( $r );
+
+        if ( ! empty( $bookmarks ) ) {
+            if ( ! empty( $r['title_li'] ) ) {
+                $output .= str_replace(
+                    array( '%id', '%class' ),
+                    array( "linkcat-" . $r['category'], $r['class'] ),
+                    $r['category_before']
+                );
+                $output .= $r['title_before'];
+                $output .= $r['title_li'];
+                $output .= $r['title_after'];
+                $output .= "\n\t<ul class='xoxo blogroll'>\n";
+                $output .= _walk_bookmarks( $bookmarks, $r );
+                $output .= "\n\t</ul>\n";
+                $output .= $r['category_after'] . "\n";
             } else {
-                $output .= _walk_bookmarks($bookmarks, $r);
+                $output .= _walk_bookmarks( $bookmarks, $r );
             }
         }
@@ -277 +301 @@
      * @since 2.5.0
      *
-     * @param string $output The HTML list of bookmarks.
+     * @param string $html The HTML list of bookmarks.
      */
-    $output = apply_filters( 'wp_list_bookmarks', $output );
-
-    if ( !$echo )
-        return $output;
-    echo $output;
+    $html = apply_filters( 'wp_list_bookmarks', $output );
+
+    if ( ! $r['echo'] ) {
+        return $html;
+    }
+    echo $html;
 }