Hi,

it would be very useful to have an option to also get notified of comments that still need to be approved.

Cheers, Fons.

CommentFileSizeAuthor
#9 notify_for_unapproved_comments.patch2.35 KBsinasalek
#3 notify_for_unapproved_comments.patch1.3 KBsinasalek

Comments

sinasalek’s picture

sinasalek commented

Yes indeed, i got confused. I was expecting to receive notification for new comments just like contents but i didn't!. So how can i know when someone posts a new comment on my personal site that needs my approval?

Slovak’s picture

Slovak commented

Quick fix details at http://peterpetrik.com/blog/2009/12/unapproved-comments-notification-wit...

sinasalek’s picture

sinasalek commented
StatusFileSize
new notify_for_unapproved_comments.patch1.3 KB

Thanks @slovak,
I created a patch out of the Quick fix and also added a condition.
Current patch sends notification for unapproved comments only when the user has "administer comments" permission.

sinasalek’s picture

sinasalek commented
Status: Active » Needs review
matt2000’s picture

matt2000 commented
Status: Needs review » Needs work

Thanks for the patch! However, I really think we need to designate in the out which comments are published & which are not, like we do with nodes (see line 479 ff.)

sinasalek’s picture

sinasalek commented

IMHO administrators should receive notification for all comments, because they can also see unapproved comments. I don't think there is any administrator who does not want to receive notification for unapproved comments.
What do you have in mind? if there was any discussion about nodes before can you send me its link?

matt2000’s picture

matt2000 commented

No, I just mean that the in the email that gets sent, it should say 'UNPUBLISHED' next to unapproved comments so the admin can tell the difference. Especially important for sites with multiple admins. Again, look at the code, see how it's done with nodes.

sinasalek’s picture

sinasalek commented

Ah, i see your point now. I'm preparing the second patch.

sinasalek’s picture

sinasalek commented
StatusFileSize
new notify_for_unapproved_comments.patch2.35 KB

Patch update to cover comment #7 suggestion.
It appears like this on email now: 

1 new comment attached to Research posted by admin: Healthy food
   1. [Unpublished] afdas saf saf sa by Ahmad
     http://sina.salek.ws/content/healthy-food#comment-199
sinasalek’s picture

sinasalek commented
Status: Needs work » Needs review
TriAn’s picture

TriAn commented

I thinck this guide will help:
http://www.jasonhooker.com/configure-drupal-notification-on-new-comments

So it helped me with unpublished comments notify

sinasalek’s picture

sinasalek commented

@matt2000 waiting for your respond

matt2000’s picture

matt2000 commented
Assigned: Unassigned » matt2000

New dev release has a possible solution for this feature request. There's no UI or optional settings; it simply includes unpublished comments for users with 'administer comments' permission. Please test.

matt2000’s picture

matt2000 commented
Status: Needs review » Fixed

Debugged the code from #9, and prepped for upcoming release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.