Project:Comment notify
Version:6.x-1.2
Component:Miscellaneous
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed (duplicate)

Issue Summary

Only site administrator receives notifications. Other registered users and anonymous users don't.

Any idea?

Comments

#1

I have changed default for anon and reg users to all comments and it works now. ??? Shouldn't it work when default is no notification and when per individual post is set to yes?

#2

Please provide more information.

1. Install Drupal core
2. Install comment notify
3. Configure x, y
4. Configure z

Expected results:
Foo

Actual results:
Bar

At this point I don't understand what you mean.

#3

Please accept my apologies for not being clear enough.

1. drupal 6.13
2. comment notify 6.x-1.2
3. Available subscription modes: only All comments checked
4. Default state for the notification selection box for anonymous users: no notifications
5. The same for registered users
6. permissions: anonymous and registered users can subscribe to comments but cannot administer comment notify

Registered or anonymous user post comment and subscribe to follow up comments.

Expected results:
He receives notification when new comment is posted. I as subscribed admin also receive notification.

Actual results:
He does not receive notification when new comment is posted. I as subscribed admin did receive notification.

Than I changed default state for the notification selection box to All comments, and after that if user subscribes - notification is sent to him for the follow up comments.
It would be great if users could receive notifications for subscribed nodes even when default state for the notification selection box is set to no notifications. Isn't that purpose of "default"? It is default not absolute.

#4

Yes, this does seem like quite a bug.

#5

Can you try with 6.x-1.x-dev and see if that fixes the problem?

#6

There was a pretty critical bug in #617070: Doesn't record subscription when only one subscription type is enabled (and lots more tests) which I just fixed. I think this might be related. Can you try updating to the latest 6.x-1.x-dev and see if that fixes it?

If so, I will make a new release.

#7

Status:active» postponed (maintainer needs more info)

Also, I just tested the steps you laid out in #3 and it worked OK. So, either 6.x-1.x-dev has the right fix for this or I will need more information to debug this particular problem.

Thanks for your persistence and patience on fixing this issue.

#8

Thanks for the efforts. I'm pretty busy right now so can't test it now. I will do it on Wednesday and let you know the results.

#9

Tested this today, using:

- Drupal 6.14
- Comment Notify 6.x-dev (2009-Nov-08)
- Available subscription modes: All comments, Replies to my comment
- Default state for the notification selection box for anonymous users: no notifications
- Default state for the notification selection box for anonymous users: Replies to my comment
- Permissions: anonymous users can't subscribe / registered users can subscribe

Node authors receive e-mails when new comments are posted to their nodes.

However, comment authors don't receive e-mails when 1) a reply to their comment is posted or 2) a new comment is posted under the node they commented on (and the comment selected 'all comments' under 'Notify me when new comments are posted')

#10

I have now tested it with settings as described in #3 and it works for me.

#11

Hi,
I was playing around with this module and also was not able to make emails working for anonymous users. I did some debugging and found that line 445 in comment_notify.module:
if ($mail != $comment_mail && !in_array($mail, $sent_to) && $alert->uid != $comment->uid) {
causes this issue.
$alert->uid is the user id who posted original comment, for anonymous user it is 0
$comment->uid is the id of the current user, for anonymous user it is 0 too.
That means the code will never go into this if to send an email if anonymous user is commenting anonymous user's comments.
I did not do extensive testing so do not blame me if I am wrong. I just have feeling that this line of code needs some attention.
I am using Drupal 6.14 and Comments Notify 6.x-1.3

#12

Status:postponed (maintainer needs more info)» closed (duplicate)

@mp-78 - That looks like #533856: Not sending e-mails.

#13

Yes, exactly. Sorry, I should have checked this first...