[wp-trac] Re: [WordPress Trac] #8091: "Parent" Comment Class

WordPress Trac wp-trac at lists.automattic.com
Fri Nov 21 12:37:47 GMT 2008


#8091: "Parent" Comment Class
--------------------------------------+-------------------------------------
 Reporter:  ThemeShaper               |        Owner:          
     Type:  enhancement               |       Status:  reopened
 Priority:  normal                    |    Milestone:  2.7     
Component:  Comments                  |      Version:  2.7     
 Severity:  normal                    |   Resolution:          
 Keywords:  has-patch Walker_Comment  |  
--------------------------------------+-------------------------------------
Changes (by filosofo):

  * keywords:  needss-patch Walker_Comment => has-patch Walker_Comment

Comment:

 Patch consistent_start_el_args.diff fixes this on two levels:
  * It checks to make sure that {{{$args[0]}}} is actually an array.  The
 empty check was pointless, since it should never be empty.
  * If fixes a more fundamental problem: {{{walk_page_tree()}}} calls the
 "walk" method with arguments in an inconsistent order.  One of these is
 not like the others in argument order, and it's the "start_el" method for
 {{{Walker_Page}}} (defined on line 1155 of wp-includes/classes.php):

 {{{ wp-admin/includes/template.php:412: function start_el(&$output,
 $category, $depth, $args) { }}}
 {{{ wp-includes/comment-template.php:1151:      function
 start_el(&$output, $comment, $depth, $args) { }}}
 {{{ wp-includes/classes.php:782:        function start_el(&$output)  {}
 }}}
 '''{{{ wp-includes/classes.php:1155:    function start_el(&$output, $page,
 $depth, $current_page, $args) { }}}'''
 {{{ wp-includes/classes.php:1233:       function start_el(&$output, $page,
 $depth, $args) { }}}
 {{{ wp-includes/classes.php:1310:       function start_el(&$output,
 $category, $depth, $args) { }}}
 {{{ wp-includes/classes.php:1425:       function start_el(&$output,
 $category, $depth, $args) { }}}

-- 
Ticket URL: <http://trac.wordpress.org/ticket/8091#comment:12>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list