WordPress.org
Get WordPress

Make WordPress Core

Changeset 7770


Ignore:
Timestamp:
04/22/2008 09:06:00 PM (16 years ago)
Author:
ryan
Message:

Convert Walker classes to pass as reference. Props mdawaffe. fixes #6796 for 2.5

Location:
branches/2.5
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • branches/2.5/wp-admin/edit-form-advanced.php

    r7762 r7770  
    242242<div id="categories-all" class="ui-tabs-panel">
    243243    <ul id="categorychecklist" class="list:category categorychecklist form-no-clear">
    244         <?php dropdown_categories( 0, 0, $popular_ids ); ?>
     244        <?php wp_category_checklist($post_ID) ?>
    245245    </ul>
    246246</div>

  • branches/2.5/wp-admin/edit-link-form.php

    r7385 r7770  
    119119<div id="categories-all" class="ui-tabs-panel">
    120120    <ul id="categorychecklist" class="list:category categorychecklist form-no-clear">
    121         <?php dropdown_link_categories(); ?>
     121        <?php wp_link_category_checklist($link_id); ?>
    122122    </ul>
    123123</div>

  • branches/2.5/wp-admin/includes/template.php

    r7767 r7770  
    114114
    115115//
    116 // Nasty Category Stuff
     116// Category Checklists
    117117//
    118118
    119 function sort_cats( $cat1, $cat2 ) {
    120     if ( $cat1['checked'] || $cat2['checked'] )
    121         return ( $cat1['checked'] && !$cat2['checked'] ) ? -1 : 1;
     119// Deprecated. Use wp_link_category_checklist
     120function dropdown_categories( $default = 0, $parent = 0, $popular_ids = array() ) {
     121    global $post_ID;
     122    wp_category_checklist($post_ID);
     123}
     124
     125class Walker_Category_Checklist extends Walker {
     126    var $tree_type = 'category';
     127    var $db_fields = array ('parent' => 'parent', 'id' => 'term_id'); //TODO: decouple this
     128
     129    function start_lvl(&$output, $depth, $args) {
     130        $indent = str_repeat("\t", $depth);
     131        $output .= "$indent<ul class='children'>\n";
     132    }
     133
     134    function end_lvl(&$output, $depth, $args) {
     135        $indent = str_repeat("\t", $depth);
     136        $output .= "$indent</ul>\n";
     137    }
     138
     139    function start_el(&$output, $category, $depth, $args) {
     140        extract($args);
     141
     142        $class = in_array( $category->term_id, $popular_cats ) ? ' class="popular-category"' : '';
     143        $output .= "\n<li id='category-$category->term_id'$class>" . '<label for="in-category-' . $category->term_id . '" class="selectit"><input value="' . $category->term_id . '" type="checkbox" name="post_category[]" id="in-category-' . $category->term_id . '"' . (in_array( $category->term_id, $selected_cats ) ? ' checked="checked"' : "" ) . '/> ' . wp_specialchars( apply_filters('the_category', $category->name )) . '</label>';
     144    }
     145
     146    function end_el(&$output, $category, $depth, $args) {
     147        $output .= "</li>\n";
     148    }
     149}
     150
     151function wp_category_checklist( $post_id = 0 ) {
     152    $walker = new Walker_Category_Checklist;
     153
     154    $args = array();
     155   
     156    if ( $post_id )
     157        $args['selected_cats'] = wp_get_post_categories($post_id);
    122158    else
    123         return strcasecmp( $cat1['cat_name'], $cat2['cat_name'] );
    124 }
    125 
    126 function wp_set_checked_post_categories( $default = 0 ) {
    127     global $post_ID, $checked_categories;
    128 
    129     if ( empty($checked_categories) ) {
    130         if ( $post_ID ) {
    131             $checked_categories = wp_get_post_categories($post_ID);
    132 
    133             if ( count( $checked_categories ) == 0 ) {
    134                 // No selected categories, strange
    135             $checked_categories[] = $default;
    136             }
    137         } else {
    138             $checked_categories[] = $default;
    139         }
    140     }
    141 
    142 }
    143 function get_nested_categories( $default = 0, $parent = 0 ) {
    144     global $checked_categories;
    145 
    146     wp_set_checked_post_categories( $default = 0 );
    147 
    148     if ( is_object($parent) ) { // Hack: if passed a category object, will return nested cats with parent as root
    149         $root = array(
    150             'children' => get_nested_categories( $default, $parent->term_id ),
    151             'cat_ID' => $parent->term_id,
    152             'checked' => in_array( $parent->term_id, $checked_categories ),
    153             'cat_name' => get_the_category_by_ID( $parent->term_id )
    154         );
    155         $result = array( $parent->term_id => $root );
    156     } else {
    157         $parent = (int) $parent;
    158 
    159         $cats = get_categories("parent=$parent&hide_empty=0&fields=ids");
    160 
    161         $result = array();
    162         if ( is_array( $cats ) ) {
    163             foreach ( $cats as $cat ) {
    164                 $result[$cat]['children'] = get_nested_categories( $default, $cat );
    165                 $result[$cat]['cat_ID'] = $cat;
    166                 $result[$cat]['checked'] = in_array( $cat, $checked_categories );
    167                 $result[$cat]['cat_name'] = get_the_category_by_ID( $cat );
    168             }
    169         }
    170     }
    171 
    172     $result = apply_filters('get_nested_categories', $result);
    173     usort( $result, 'sort_cats' );
    174 
    175     return $result;
    176 }
    177 
    178 function write_nested_categories( $categories, $popular_ids = array() ) {
    179     foreach ( $categories as $category ) {
    180         $class = in_array( $category['cat_ID'], $popular_ids ) ? ' class="popular-category"' : '';
    181         echo "\n", "<li id='category-$category[cat_ID]'$class>", '<label for="in-category-', $category['cat_ID'], '" class="selectit"><input value="', $category['cat_ID'], '" type="checkbox" name="post_category[]" id="in-category-', $category['cat_ID'], '"', ($category['checked'] ? ' checked="checked"' : "" ), '/> ', wp_specialchars( apply_filters('the_category', $category['cat_name'] )), '</label>';
    182 
    183         if ( $category['children'] ) {
    184             echo "\n<ul>";
    185             write_nested_categories( $category['children'] );
    186             echo "\n</ul>";
    187         }
    188         echo '</li>';
    189     }
    190 }
    191 
    192 function dropdown_categories( $default = 0, $parent = 0, $popular_ids = array() ) {
    193     write_nested_categories( get_nested_categories( $default, $parent ), $popular_ids );
     159        $args['selected_cats'] = array();
     160    $args['popular_cats'] = get_terms( 'category', array( 'fields' => 'ids', 'orderby' => 'count', 'order' => 'DESC', 'number' => 10 ) );
     161    $categories = get_categories('get=all');
     162    $args = array($categories, 0, $args);
     163    $output = call_user_func_array(array(&$walker, 'walk'), $args);
     164
     165    echo $output;
    194166}
    195167
    … …  
    215187}
    216188
     189// Deprecated. Use wp_link_category_checklist
    217190function dropdown_link_categories( $default = 0 ) {
    218191    global $link_id;
    219192
     193    wp_link_category_checklist($link_id);
     194}
     195
     196function wp_link_category_checklist( $link_id = 0 ) {
    220197    if ( $link_id ) {
    221198        $checked_categories = wp_get_link_cats($link_id);

  • branches/2.5/wp-admin/post.php

    r7760 r7770  
    3636    } elseif (!empty($referredby) && $referredby != $referer) {
    3737        $location = $_POST['referredby'];
     38        $location = remove_query_arg('_wp_original_http_referer', $location);
    3839        if ( $_POST['referredby'] == 'redo' )
    3940            $location = get_permalink( $post_ID );

  • branches/2.5/wp-includes/classes.php

    r7551 r7770  
    396396
    397397    //abstract callbacks
    398     function start_lvl($output) { return $output; }
    399     function end_lvl($output)   { return $output; }
    400     function start_el($output)  { return $output; }
    401     function end_el($output)    { return $output; }
     398    function start_lvl(&$output) {}
     399    function end_lvl(&$output)   {}
     400    function start_el(&$output)  {}
     401    function end_el(&$output)    {}
    402402
    403403    /*
    … …  
    405405     * otherwise, display the element and its children
    406406     */
    407     function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, $output ) {
    408 
    409         if ( !$element)
    410             return $output;
     407    function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {
     408
     409        if ( !$element )
     410            return;
    411411
    412412        $id_field = $this->db_fields['id'];
    … …  
    414414
    415415        //display this element
    416         $cb_args = array_merge( array($output, $element, $depth), $args);
    417         $output = call_user_func_array(array(&$this, 'start_el'), $cb_args);
     416        $cb_args = array_merge( array(&$output, $element, $depth), $args);
     417        call_user_func_array(array(&$this, 'start_el'), $cb_args);
    418418
    419419        if ( $max_depth == 0 ||
    … …  
    428428                        $newlevel = true;
    429429                        //start the child delimiter
    430                         $cb_args = array_merge( array($output, $depth), $args);
    431                         $output = call_user_func_array(array(&$this, 'start_lvl'), $cb_args);
     430                        $cb_args = array_merge( array(&$output, $depth), $args);
     431                        call_user_func_array(array(&$this, 'start_lvl'), $cb_args);
    432432                    }
    433433
    434434                    array_splice( $children_elements, $i, 1 );
    435                     $output = $this->display_element( $child, $children_elements, $max_depth, $depth + 1, $args, $output );
     435                    $this->display_element( $child, $children_elements, $max_depth, $depth + 1, $args, $output );
    436436                    $i = -1;
    437437                }
    … …  
    441441        if ( isset($newlevel) && $newlevel ){
    442442            //end the child delimiter
    443             $cb_args = array_merge( array($output, $depth), $args);
    444             $output = call_user_func_array(array(&$this, 'end_lvl'), $cb_args);
     443            $cb_args = array_merge( array(&$output, $depth), $args);
     444            call_user_func_array(array(&$this, 'end_lvl'), $cb_args);
    445445        }
    446446
    447447        //end this element
    448         $cb_args = array_merge( array($output, $element, $depth), $args);
    449         $output = call_user_func_array(array(&$this, 'end_el'), $cb_args);
    450 
    451         return $output;
     448        $cb_args = array_merge( array(&$output, $element, $depth), $args);
     449        call_user_func_array(array(&$this, 'end_el'), $cb_args);
    452450    }
    453451
    … …  
    477475            $empty_array = array();
    478476            foreach ( $elements as $e )
    479                 $output = $this->display_element( $e, $empty_array, 1, 0, $args, $output );
     477                $this->display_element( $e, $empty_array, 1, 0, $args, $output );
    480478            return $output;
    481479        }
    … …  
    513511
    514512        foreach ( $top_level_elements as $e )
    515             $output = $this->display_element( $e, $children_elements, $max_depth, 0, $args, $output );
     513            $this->display_element( $e, $children_elements, $max_depth, 0, $args, $output );
    516514
    517515        /*
    … …  
    522520            $empty_array = array();
    523521            foreach ( $children_elements as $orphan_e )
    524                 $output = $this->display_element( $orphan_e, $empty_array, 1, 0, $args, $output );
     522                $this->display_element( $orphan_e, $empty_array, 1, 0, $args, $output );
    525523         }
    526524         return $output;
    … …  
    532530    var $db_fields = array ('parent' => 'post_parent', 'id' => 'ID'); //TODO: decouple this
    533531
    534     function start_lvl($output, $depth) {
     532    function start_lvl(&$output, $depth) {
    535533        $indent = str_repeat("\t", $depth);
    536534        $output .= "\n$indent<ul>\n";
    537         return $output;
    538     }
    539 
    540     function end_lvl($output, $depth) {
     535    }
     536
     537    function end_lvl(&$output, $depth) {
    541538        $indent = str_repeat("\t", $depth);
    542539        $output .= "$indent</ul>\n";
    543         return $output;
    544     }
    545 
    546     function start_el($output, $page, $depth, $current_page, $args) {
     540    }
     541
     542    function start_el(&$output, $page, $depth, $current_page, $args) {
    547543        if ( $depth )
    548544            $indent = str_repeat("\t", $depth);
    … …  
    572568            $output .= " " . mysql2date($date_format, $time);
    573569        }
    574 
    575         return $output;
    576     }
    577 
    578     function end_el($output, $page, $depth) {
     570    }
     571
     572    function end_el(&$output, $page, $depth) {
    579573        $output .= "</li>\n";
    580 
    581         return $output;
    582574    }
    583575
    … …  
    588580    var $db_fields = array ('parent' => 'post_parent', 'id' => 'ID'); //TODO: decouple this
    589581
    590     function start_el($output, $page, $depth, $args) {
    591                 $pad = str_repeat('&nbsp;', $depth * 3);
    592 
    593                 $output .= "\t<option value=\"$page->ID\"";
    594                 if ( $page->ID == $args['selected'] )
    595                                 $output .= ' selected="selected"';
    596                 $output .= '>';
    597                 $title = wp_specialchars($page->post_title);
    598                 $output .= "$pad$title";
    599                 $output .= "</option>\n";
    600 
    601                 return $output;
     582    function start_el(&$output, $page, $depth, $args) {
     583        $pad = str_repeat('&nbsp;', $depth * 3);
     584
     585        $output .= "\t<option value=\"$page->ID\"";
     586        if ( $page->ID == $args['selected'] )
     587            $output .= ' selected="selected"';
     588        $output .= '>';
     589        $title = wp_specialchars($page->post_title);
     590        $output .= "$pad$title";
     591        $output .= "</option>\n";
    602592    }
    603593}
    … …  
    607597    var $db_fields = array ('parent' => 'parent', 'id' => 'term_id'); //TODO: decouple this
    608598
    609     function start_lvl($output, $depth, $args) {
     599    function start_lvl(&$output, $depth, $args) {
    610600        if ( 'list' != $args['style'] )
    611             return $output;
     601            return;
    612602
    613603        $indent = str_repeat("\t", $depth);
    614604        $output .= "$indent<ul class='children'>\n";
    615         return $output;
    616     }
    617 
    618     function end_lvl($output, $depth, $args) {
     605    }
     606
     607    function end_lvl(&$output, $depth, $args) {
    619608        if ( 'list' != $args['style'] )
    620             return $output;
     609            return;
    621610
    622611        $indent = str_repeat("\t", $depth);
    623612        $output .= "$indent</ul>\n";
    624         return $output;
    625     }
    626 
    627     function start_el($output, $category, $depth, $args) {
     613    }
     614
     615    function start_el(&$output, $category, $depth, $args) {
    628616        extract($args);
    629617
    … …  
    688676            $output .= "\t$link<br />\n";
    689677        }
    690 
    691         return $output;
    692     }
    693 
    694     function end_el($output, $page, $depth, $args) {
     678    }
     679
     680    function end_el(&$output, $page, $depth, $args) {
    695681        if ( 'list' != $args['style'] )
    696             return $output;
     682            return;
    697683
    698684        $output .= "</li>\n";
    699         return $output;
    700685    }
    701686
    … …  
    706691    var $db_fields = array ('parent' => 'parent', 'id' => 'term_id'); //TODO: decouple this
    707692
    708     function start_el($output, $category, $depth, $args) {
     693    function start_el(&$output, $category, $depth, $args) {
    709694        $pad = str_repeat('&nbsp;', $depth * 3);
    710695
    … …  
    722707        }
    723708        $output .= "</option>\n";
    724 
    725         return $output;
    726709    }
    727710}
Note: See TracChangeset for help on using the changeset viewer.