2 notification sent Issue (if comments auto-approval is turned off)

endevver / mt-plugin-comment-subscribe

A plugin for Movable Type that keeps readers updated when people submit follow up comments.

5 stars 1 forks source link

Open andrewtrell opened 14 years ago

andrewtrell commented 14 years ago

That is, if "Immediately approve comments from" is set to "No one"..

The notifications are still sent out, even if the comment has not been approved. Then, if you approve it, the notifications are all sent out again.

Looks like just a small, but important bug.

Blogosfere commented 13 years ago

hello Andrew & Byrne,

this should fix the notification bug:

Change row 15 at lib/CommentSubscribe/Plugin.pm , from

if ( $obj->visible || $obj->junk_status == 1 ) {

if ( $obj->visible ) {

Does it work for you? :-)

Cheers

mihaibocsaru commented 11 years ago

I confirm that the above patch fixes the double notification issue.