I was looking for some of the features provided by this module and i couldn't find elsewhere...
But i didn't realize how the module works at the first look; it is meant to notify comment owners but not the node owner.

I understand that this was a choice and that is meant to make users come back to the site when they start to participate writing a comment.
But i think it will be so much better if we have the chance to notify the node owner too, 'cause he's the one the comments are related to, and he's the one who first want to know about comments on his post. Isn't?

So, i'm asking if you can evaluate this request, the module is great even if i couldn't choose to have automatic notification because it didn't work and i had to keep the "Let registered users select notification on a node basis" box checked to make it work.
Thank you so much,
Da.
http://inkscapegallery.net

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

Well, I'd like to keep this module as simple as possible and as focused on its purpose as possible. I'm not sure how Cristoph feels. There is the subscriptions module which already offers this feature. Maybe you could try that out?

planctus’s picture

i handled this with workflow_ng, but i needed an extra module to do this.
You're right choosing to maintain the module light and simple, but i was wondering if comment_notify could be the "comment notifications" handler module for drupal, instead of using various modules to add common features to the comment module.
Thanks,
Da.

greggles’s picture

Sure - I think that subscriptions will become that "one module" to manage all mails based on events. I think that at some point it could make sense for comment_notify to become a lighter weight UI on top of subscriptions.

Junyor’s picture

FWIW, I'd like to see this functionality as well. I'm using comment_notify for a personal weblog where I'm the only registered user. It seems a waste to add a full subscription/notification framework just so I can get e-mails when users reply to my posts.

Right now, I'm looking at commentmail as an alternative for this functionality, but I'd rather not add another module.

greggles’s picture

@Junyor - what if we went the other way? I've got code which uses the Subscriptions module as the backend for comment_notify. What if we migrate from "comment_notify handles everything" to "comment_notify is just a UI on top of subscriptions"?

Then we get rid of your major complaint, right? And we also get to leverage the many great features of subscriptions without having to re-implement them here.

Junyor’s picture

Yes, I suppose that would work. You're talking about Subscriptions 2.0, right? I tried beta 10 last night, but couldn't get it working.

greggles’s picture

Yes, Subscriptions 2.0 and yes, I haven't put more effort into it since it is still beta quality ;)

benkenney’s picture

I'd like this feature as well. I think it would also be helpful in the forums. Sometimes a user would create a new forum topic and then weeks/months later a reply would come but the original author wouldn't necessarily be notified.

urbanfalcon’s picture

urbanfalcon’s picture

Status: Active » Needs review
FileSize
10.89 KB

Here's a simple patch which allows node posters to receive notifications when comments are posted. It is a very small but powerful addition and should help keep site energy up. I know that comment_notify is meant to keep visitors coming back; keeping the original poster involved in any discourse has the potential to do just that.

ssace’s picture

Is there a patch for the latest ver 1.6 of comment_notify?

funana’s picture

+1

this has to go into the next comment notify version! Or into Subscriptions 2.0, whatever...

greggles’s picture

@funana - it is already in subscriptions. I'm using it on 2 sites currently.

ssace’s picture

The reason I prefer comment_notify is that it's light-weight. Our forums are usually very informal short conversations. The subscription
module would be overkill for our needs. I hope comment_notify is updated soon to include notice to node authors.

sdsheridan’s picture

Version: 5.x-1.4 » 5.x-1.6

I patched comment notify (5.x-1.6) with this and it's working brilliantly! Thanks! I agree, as I don't use subscriptions either. This really should be (IMHO) a feature of comment notify.

funana’s picture

@greggles:

I once tested subscriptions and it was not very performance friendly. Plus, I want to use it on a site with freetagging taxonomy with over 35.000 terms, which produced a lot of headache with subscriptions. That's why I want to keep Comment Notify... I think I will patch it too.

premanup’s picture

Thank you! It's working for me!
Yes, this realy should be a feature of module.

wwwoliondorcom’s picture

Hi,

I'm using Drigg, the super-great digg-like drupal module, and I would also like the node owner to be notified when somebody comment his News.

Right now I'm using:

Comment_subscribe to let people who send a comment get a notification for replies to their comment
and
Comment Mail for the admin (me) to be notified when somebody send a comment on the website

As this is working perfectly I don't want to change the whole thing, but I would like the node author / owner to be notified when somebody comment his news, so what would be the best option to add this function ?

Thanks a lot.

premanup’s picture

Version: 5.x-1.6 » 5.x-2.1

May I ask you rewrite this patch for verison 2.1 ?
Thank you!

wwwoliondorcom’s picture

I found a better solution, use WATCHER module, the ultimate notification module... (really perfect, but only for registered users)

greggles’s picture

Which is...hilarious because one of the major points of comment_notify is to work for anonymous...oh well.

greggles’s picture

Assigned: Unassigned » greggles
FileSize
8.05 KB

So, I've finally "seen the light" and agree that this would be a handy feature for comment_notify. The current patch is quite interesting - it tries to do a little more than just adding this one feature but overall it looks pretty good to me.

A couple changes compared to the old patch:
I made it so that authors must opt-in to get the notifications.
I left the subject the same as it presently is - if we change that it should be its own issue.
I also removed a couple of random changes and left in a couple of random changes.

I plan to commit this today and would love a review before doing it...or after doing it ;)

greggles’s picture

Here's a re-roll...

greggles’s picture

Version: 5.x-2.1 » 5.x-2.x-dev
Status: Needs review » Fixed

This has now been added to 5.x-2.x and 6.x-1.x.

Thanks, for the ideas everyone.

greggles’s picture

I had a syntax error in the 6.x version for a second...but have now fixed it.

CompShack’s picture

I just updated to the newest version and was looking forward to this new feature but it looks like it is not available for anonymous users, is it?

UPDATE: After thinking about this for a little bit, I think i know why this isn't implemented for anonymous users. Well, in my case http://www.CompShack.com/forum users can Ask questions without registering BUT the problem is that creating a new node page doesn't have any email field for guest to enter their email address to subscribe to the node they just created.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.