Node authors not geting notification unless they save their account page

Same here.

+ same with dev version 1.x

i think it can be critical, cause it is default option for node author.

subscribed.

+1
I think it can be critical too.

I think here (http://drupal.org/node/849780) is discussed same problem.

Hi guys, here a solution of the problem!

Comment notify module sends email to the node's author after checking database table "comment_notify_user_settings". This table contains user ids and their settings for notifying. Module fills this table during "comment" and "user" hooks. And sometimes, especially new users, doesn't have their rows in the table and as a result module doesn't send mail.

My solution is very simple. It checks variable "node_notify_default_mailalert" if "comment_notify_user_settings" doesn't have the row with current user and depends on it's value sends or doesn't send mail.

Here my patch. Please, test it.
Sorry for my English.

i will check it soon! spasibo

Better status.

I fixed this a slightly different way in #774826: node_notify_default_mailalert not respected.

I've marked that as a duplicate of this. Could you folks test out the attached patch?

Hmm. This is a critical I think.

However, without actually reviewing the code it appears to me that it doesn't work.

I created a node as one user, commented on it as another user, had the appropriate checkbox set in admin/settings/comment_notify. No email notification was generated to the node author.

Thanks for the excellent work on this everybody.

it's really good.
a finely, delicate patch.

I question my report in #9, so if it's not the same as other people's experience should be disregarded.

I've had this deployed on warmshowers.org for several months and it works as advertised. I have found nothing wrong with it.

Nice work, and thanks.

Fixed - thanks Randy http://drupalcode.org/project/comment_notify.git/commit/c95c1f2