Clean up wp_list_authors(). See #10329

git-svn-id: https://develop.svn.wordpress.org/trunk@15620 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
scribu 2010-09-15 19:52:25 +00:00
parent 30581c2d4d
commit 322e323b26
1 changed files with 56 additions and 63 deletions

View File

@ -257,7 +257,7 @@ function get_author_posts_url($author_id, $author_nicename = '') {
* echoing.</li> * echoing.</li>
* <li>style (string) ('list'): Whether to display list of authors in list form * <li>style (string) ('list'): Whether to display list of authors in list form
* or as a string.</li> * or as a string.</li>
* <li>html (bool) (true): Whether to list the items in html for or plaintext. * <li>html (bool) (true): Whether to list the items in html form or plaintext.
* </li> * </li>
* </ul> * </ul>
* *
@ -270,101 +270,94 @@ function wp_list_authors($args = '') {
global $wpdb; global $wpdb;
$defaults = array( $defaults = array(
'orderby' => 'name', 'order' => 'ASC', 'number' => '',
'optioncount' => false, 'exclude_admin' => true, 'optioncount' => false, 'exclude_admin' => true,
'show_fullname' => false, 'hide_empty' => true, 'show_fullname' => false, 'hide_empty' => true,
'feed' => '', 'feed_image' => '', 'feed_type' => '', 'echo' => true, 'feed' => '', 'feed_image' => '', 'feed_type' => '', 'echo' => true,
'style' => 'list', 'html' => true 'style' => 'list', 'html' => true
); );
$r = wp_parse_args( $args, $defaults ); $args = wp_parse_args( $args, $defaults );
extract($r, EXTR_SKIP); extract( $args, EXTR_SKIP );
$return = ''; $return = '';
/** @todo Move select to get_authors(). */ $authors = get_users( wp_array_slice_assoc( $args, array( 'orderby', 'order', 'number' ) ) );
$users = get_users();
$author_ids = array();
foreach ( (array) $users as $user )
$author_ids[] = $user->user_id;
if ( count($author_ids) > 0 ) {
$author_ids = implode(',', $author_ids );
$authors = $wpdb->get_results( "SELECT ID, user_nicename from $wpdb->users WHERE ID IN($author_ids) " . ($exclude_admin ? "AND user_login <> 'admin' " : '') . "ORDER BY display_name" );
} else {
$authors = array();
}
$author_count = array(); $author_count = array();
foreach ( (array) $wpdb->get_results("SELECT DISTINCT post_author, COUNT(ID) AS count FROM $wpdb->posts WHERE post_type = 'post' AND " . get_private_posts_cap_sql( 'post' ) . " GROUP BY post_author") as $row ) foreach ( (array) $wpdb->get_results("SELECT DISTINCT post_author, COUNT(ID) AS count FROM $wpdb->posts WHERE post_type = 'post' AND " . get_private_posts_cap_sql( 'post' ) . " GROUP BY post_author") as $row )
$author_count[$row->post_author] = $row->count; $author_count[$row->post_author] = $row->count;
foreach ( (array) $authors as $author ) { foreach ( $authors as $author ) {
if ( $exclude_admin && 'admin' == $author->display_name )
continue;
$posts = isset( $author_count[$author->ID] ) ? $author_count[$author->ID] : 0;
if ( !$posts && $hide_empty )
continue;
$link = ''; $link = '';
$author = get_userdata( $author->ID ); if ( $show_fullname && $author->first_name && $author->last_name )
$posts = (isset($author_count[$author->ID])) ? $author_count[$author->ID] : 0;
$name = $author->display_name;
if ( $show_fullname && ($author->first_name != '' && $author->last_name != '') )
$name = "$author->first_name $author->last_name"; $name = "$author->first_name $author->last_name";
else
$name = $author->display_name;
if( !$html ) { if ( !$html ) {
if ( $posts == 0 ) { $return .= $name . ', ';
if ( ! $hide_empty )
$return .= $name . ', ';
} else
$return .= $name . ', ';
// No need to go further to process HTML. continue; // No need to go further to process HTML.
continue;
} }
if ( !($posts == 0 && $hide_empty) && 'list' == $style ) if ( 'list' == $style ) {
$return .= '<li>'; $return .= '<li>';
if ( $posts == 0 ) { }
if ( ! $hide_empty )
$link = $name;
} else {
$link = '<a href="' . get_author_posts_url($author->ID, $author->user_nicename) . '" title="' . esc_attr( sprintf(__("Posts by %s"), $author->display_name) ) . '">' . $name . '</a>';
if ( (! empty($feed_image)) || (! empty($feed)) ) { $link = '<a href="' . get_author_posts_url( $author->ID, $author->user_nicename ) . '" title="' . esc_attr( sprintf(__("Posts by %s"), $author->display_name) ) . '">' . $name . '</a>';
$link .= ' ';
if (empty($feed_image))
$link .= '(';
$link .= '<a href="' . get_author_feed_link($author->ID) . '"';
if ( !empty($feed) ) { if ( !empty( $feed_image ) || !empty( $feed ) ) {
$title = ' title="' . esc_attr($feed) . '"'; $link .= ' ';
$alt = ' alt="' . esc_attr($feed) . '"'; if ( empty( $feed_image ) ) {
$name = $feed; $link .= '(';
$link .= $title;
}
$link .= '>';
if ( !empty($feed_image) )
$link .= "<img src=\"" . esc_url($feed_image) . "\" style=\"border: none;\"$alt$title" . ' />';
else
$link .= $name;
$link .= '</a>';
if ( empty($feed_image) )
$link .= ')';
} }
if ( $optioncount ) $link .= '<a href="' . get_author_feed_link( $author->ID ) . '"';
$link .= ' ('. $posts . ')';
$alt = $title = '';
if ( !empty( $feed ) ) {
$title = ' title="' . esc_attr( $feed ) . '"';
$alt = ' alt="' . esc_attr( $feed ) . '"';
$name = $feed;
$link .= $title;
}
$link .= '>';
if ( !empty( $feed_image ) )
$link .= '<img src="' . esc_url( $feed_image ) . '" style="border: none;"' . $alt . $title . ' />';
else
$link .= $name;
$link .= '</a>';
if ( empty( $feed_image ) )
$link .= ')';
} }
if ( $posts || ! $hide_empty ) if ( $optioncount )
$return .= $link . ( ( 'list' == $style ) ? '</li>' : ', ' ); $link .= ' ('. $posts . ')';
$return .= $link;
$return .= ( 'list' == $style ) ? '</li>' : ', ';
} }
$return = trim($return, ', '); $return = rtrim($return, ', ');
if ( ! $echo ) if ( !$echo )
return $return; return $return;
echo $return; echo $return;
} }