Count User Posts By Type (WP-Related)

[fize4ever]

I'm trying to get this function to work, but it doesn't output anything.

Shouldn't this work?

PHP Code:

function count_user_posts_by_type($userid, $post_type='member') {

global $wpdb;

$where = get_posts_by_author_sql($post_type, TRUE, $userid);

$count = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->posts $where" );

return apply_filters('get_usernumposts', $count, $userid);

}

count_user_posts_by_type(19); 


[nemrod]

There are two problems.

First is that you've changed the default value for the $post_type argument from 'post' to 'member'. 'member' isn't a post type and so 0 will always be returned (unless you pass an argument that replaces it). Change it back to post or page or any other valid post type.

Second is that you're not actually displaying the result. The function returns an integer, but you never echo it. "echo count_user_posts_by_type(19);" will do the trick.

[fize4ever]

Originally Posted by nemrod

There are two problems.

First is that you've changed the default value for the $post_type argument from 'post' to 'member'. 'member' isn't a post type and so 0 will always be returned (unless you pass an argument that replaces it). Change it back to post or page or any other valid post type.

Second is that you're not actually displaying the result. The function returns an integer, but you never echo it. "echo count_user_posts_by_type(19);" will do the trick.

That worked, so thank you Mr. Nice Viking!

I've registered member as a custom post type, and the user of the ID value 19 has created some posts on it, and I want to count these posts.

You're right, it works for post and page, so how should I make it work with the custom post type of 'member'?

[nemrod]

I would construct the WHERE clause manually instead of using get_posts_by_author_sql() since it doesn't seem to work with 'member' for me (though that could be because I don't have the post type).
Set $where to "WHERE post_author = $userid AND post_type = '$post_type' AND (post_status = 'publish')" and see if that works.

[fize4ever]

Doesn't seem to quite work.

Here's how it looks with me:

PHP Code:

function count_user_posts_by_type($userid, $post_type='member') { 

global $wpdb; 

$where = WHERE post_author = $userid AND post_type = '$post_type' AND (post_status = 'publish');

$count = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->posts $where" ); 

return apply_filters('get_usernumposts', $count, $userid); 

} 


[nemrod]

Originally Posted by fize4ever

Doesn't seem to quite work.

Here's how it looks with me:

PHP Code:

function count_user_posts_by_type($userid, $post_type='member') { 

global $wpdb; 

$where = WHERE post_author = $userid AND post_type = '$post_type' AND (post_status = 'publish');

$count = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->posts $where" ); 

return apply_filters('get_usernumposts', $count, $userid); 

} 


I'd write it like this:

PHP Code:

function count_user_posts_by_type($userid, $post_type = 'member') {
    global $wpdb;
    $query = "SELECT COUNT(*) FROM $wpdb->posts WHERE post_author = $userid AND post_type = '$post_type'";
    $count = $wpdb->get_var($query);
    return apply_filters('get_usernumposts', $count, $userid);
}
echo count_user_posts_by_type(19); 


I merged the strings and removed the publish status since I don't know what status posts of the member type will have. You can add it back if needed. I'm not sure what the filter is supposed to do, you could probably simplify it further by removing that and simply returning $count.

[fize4ever]

Marry me.

[nemrod]

Originally Posted by fize4ever

Marry me.

Haha, I'll take it that it worked then, good.