Check capabilities/role before adding comment links to email

[GaryJ]

Currently the Trash it, Delete it, Spam it are added to comment notifcation emails, which are sent to the author.

However, the author may have had their role changed down to a lower-role (or had capability removed) since they wrote the post, and no longer would have permissions to use the links added to the email.

It may also be the case that a subscriber user was assigned as the Post Author (perhaps as a sort of Guest Author) for a post, and never had the permissions to use those links.

Can some sort of check be added, before these links be added to the email?

[solarissmoke]

Think this should be relatively easy to do, here's a patch that works for me.

[solarissmoke]

Check that a post author still has permissions to edit comments before inserting spam/trash links into notification email

[tislam100]

I tried your patch by making a plugin. Now, whenever I post a comment I get a blank page. here is the link to the plugin:

​http://pastebin.com/9zeAecXj

[solarissmoke]

Replying to tislam100:

I tried your patch by making a plugin. Now, whenever I post a comment I get a blank page.

Works fine for me. Are you using trunk?

[tislam100]

What is trunk? Did you try the plugin I made? Maybe there is a bug in the plugin. can you take a look at it?

Thank you.

[solarissmoke]

Trunk is the latest development version of Wordpress, as opposed to the latest stable version. The patch (and your plugin) will not work in Wordpress 3.0.5 because it uses the user_can() function is new to Wordpress 3.1. It works in 3.1 however.

[tislam100]

any alternative to user_can() that works in wordpress 3.0.5?
Can it be done with user role?

Thanks

[solarissmoke]

This is going off-topic for this ticket. Please lets take this to #wordpress IRC or the support forums.

[tislam100]

yeah. you are right. I am sorry.

[wonderboymusic]

Since [16223], the post_author doesn't even get the email if they don't pass user_can( $user->ID, 'edit_comment', $comment_id ) - which is the same check the proposed patch suggests.