Subscribe/Unsubscribe should be by node, not by comment

rfay - August 3, 2009 - 23:15
Project:Comment Notify
Version:6.x-1.2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

When you subscribe using comment_notify, it would be better for the subscription attached to a node, not a comment. Otherwise, if you participate in a conversation, it may be nearly impossible to unsubscribe, since you may be subscribed every time you comment.

#1

greggles - August 3, 2009 - 23:19

The underlying bug is:

1. Post 2 comments on a node where you subscribe to notifications on both
2. A new comment is posted, you get an e-mail
3. Click the unsubscribe link

Expected results:
The user never gets e-mails again from this thread.

Actual results:
One of the comments is unsubscribed, but the other remains subscribed.

The proposed solution from OP would also help solve #288726: allow users to subscribe without posting a comment.

An alternate solution would be to replace the cid in the has with the nid.

 
 

Drupal is a registered trademark of Dries Buytaert.