Unnecessary function argument for display_rows

[cojennin]

display_rows($posts = array()) in /wp-admin/includes/class-wp-posts-list-table.php (line 303) overrides display_rows() in /wp-admin/includes/class-wp-list-table.php (line 807).

display_rows($posts = array()) never uses the $posts argument because $posts is assigned $wp_query->posts later on in the function.

This argument and the if(empty($posts)) check later in the function are extraneous.

[cojennin]

removed function argument and empty check

[nacin]

The $level = 0 parameter is also useless, yes? We add an argument to a method we override but never use it. This is likely confusion stemming from _display_rows().

Patch looks good otherwise. If uploaded with a diff or patch extension, it will be color-coded here on Trac.

[wonderboymusic]

Line 1368 of ajax-actions.php:

$wp_list_table = _get_list_table('WP_Posts_List_Table');

$level gets set, and then....

Line 1382 of ajax-actions.php:

$wp_list_table->display_rows( array( get_post( $_POST['post_ID'] ) ), $level );

Uses both arguments... so, the patch would break that...