Help with coding

[drmike]

I'm looking at upgrading the "post views" plugin that we use and wanted to bounce this off and see if anyone else had a better way of doing this.

We want to not count those visitors who are logged in and are a member of the blog in question.  I'm thinking about using the get_current_user() function and then doing a check against "can they write" or whatever it's called.  If true, then don't coung the visit.

Anyone else have a better method?

Here's the current code:

Code:

$should_count = false;
switch(intval($views_options['count'])) {
case 0:
$should_count = true;
break;
case 1:
if(empty($_COOKIE[USER_COOKIE])) {
$should_count = true;
}
break;
case 2:
if(intval($user_ID) > 0) {
$should_count = true;
}
break;
}

The cookie check won't work as they could be logged in but not a member of the blog in question.

Thanks,
-drmike

[Luke]

something like:

Code:

<?php
if ( !user_can('edit_posts') ) {
     count_post_function();
}
?>

[drmike]

Yes, that's what I was talking about.

My question though was is there a better method or should I just use that one?

[Luke]

I would think my example would cut a bit of stuff out.

[drmike]

I would think my example would cut a bit of stuff out.

Agreed.  I was just wondering about other options.

[Luke]

Sure, there's lots of ways to do it.

The shorter, simpler it is, the more efficient it is though and you can't get much more simpler than that.

[drmike]

Actually I just found a function labeled is_user_member_of_blog built in.  It's in wpmu_functions.php in case anyone wants to look at it.