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

Any idea?

Comments

sedmi’s picture

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?

greggles’s picture

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.

sedmi’s picture

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.

greggles’s picture

Yes, this does seem like quite a bug.

greggles’s picture

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

greggles’s picture

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.

greggles’s picture

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.

sedmi’s picture

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.

brunodbo’s picture

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')

sedmi’s picture

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

mp-78’s picture

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

greggles’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)

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

mp-78’s picture

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