Hello everyone,
Great to have a light module that can be sued instead of the full message framework.
I want all users to have notifications from their own content (I only allow users to create blogs and forum threads) on as default so that they can receive comments from the blogs and forum threads they have initiated. I do not see that in the default options on the admin menu. I decided to change the database and make the default 1 there (I double chekced that it was still default) but when I create an user the user still gets that function off as default. not sure if I am doing something wrong? (first time I change a default on a database so I might be missing something obvious)
any idea on what I can do to change this behaviour?
Any help is greatly appreciated.
Comment | File | Size | Author |
---|---|---|---|
#11 | Picture 8.png | 19.89 KB | Elijah Lynn |
#9 | comment_notify-comment_notify.module-402988.patch | 1.54 KB | darktygur-1 |
Comments
Comment #1
gregglesThis is not possible now, so I'm marking it a feature request.
I'm not sure we should provide this because it further complicates the user interface.
Comment #2
netbjarne CreditAttribution: netbjarne commentedI agree on the feature request. Or at least, enable the "main switch" in the profile view when a user enables notification on a comment. Apparently, today its possible to select notification when posting a comment, without getting any - until you enable comment notification in the user profile. At minimum this is a usability issue.
Oh - thanks for the comment notify module by the way :)
Bjarne
Comment #3
gregglesIf that is the case it's a bug. I will try to test that specifically, but if you could post a "simplified, repeatable test case" it would be very helpful.
Comment #4
Anonymous (not verified) CreditAttribution: Anonymous commentedI'd also like to see this so new users are automatically notified of content they create.
@totocol is this the line you changed from FALSE to TRUE?
'#default_value' => isset($edit['node_notify_mailalert']) ? $edit['node_notify_mailalert'] : FALSE,
Comment #5
dragonwize CreditAttribution: dragonwize commentedI too am very interested in this feature.
I need it on a site right now so I've done:
Changed comment_notify.module line 291 to:
Changed comment_notify.module line 375 to:
But there definitely needs to be an admin option to set this as default on or off that would control these lines. A little busy right now but I will work up a proper patch for review when I get a chance.
Comment #6
wwwoliondorcom CreditAttribution: wwwoliondorcom commentedHi,
+1
I agree that this option would be great.
Thanks.
Comment #7
Elijah Lynn+100
This is definitely needed. It doesn't make sense for a admin to be able to set a "Default state for the notification selection box for registered users." if it doesn't send them by default.
Comment #8
Drupalized CreditAttribution: Drupalized commentedsubscribing
Comment #9
darktygur-1 CreditAttribution: darktygur-1 commentedHere the patch I've made for this problem. It's so simple, I don't understand why somebody else didn't already make it by now.
Comment #10
wwwoliondorcom CreditAttribution: wwwoliondorcom commentedIt seems that is now time to learn how to apply a patch ! Thank you so much !
Just two questions, does your patch also enables comment notification of comments made by the user ?
And do you know a way to enable comment notification (for posts) of all previous posts made by users before I use this patch ?
Thanks again.
Comment #11
Elijah LynnThis patch adds a checkbox to the Comment Notify admin settings page that lets you specify the default for new users. See screenshot attached.
Comment #12
Elijah LynnThanks so much buddy! This works great!
Comment #13
wwwoliondorcom CreditAttribution: wwwoliondorcom commentedAny idea to enable comment notify for all previously created users ?
Thanks.
Comment #14
greggles@wwwoliondorcom - http://drupal.org/node/252697
@darktygur - thanks, I'll try to review this and commit it soon.
Comment #15
wwwoliondorcom CreditAttribution: wwwoliondorcom commentedThank you, but I was speaking about comment notification for node creation.
I mean, all the people who previously posted a node on the website, i would like that they get notification of new messages on their node.
Possible ?
Comment #17
Elijah Lynn@Greggles - Any chance you will have time to review this soon?
Comment #18
gregglesThanks @darkTyger - now fixed http://drupal.org/cvs?commit=277488
@Elijah Lynn - asking me to review something usually makes me less likely to review. Providing your own review (which you did, but not in a detailed way) or providing simpletests for the code is likely to make me commit it.
Comment #19
Elijah Lynn@Greggles - Thanks for explaining that. I need to learn how to do SimpleTests and review stuff more so than just saying "it works", maybe at DIWD or one of the Meetups I will learn that. It is a bit over my head atm.
Anyways, thanks a bunch for committing this!
Comment #20
gregglesSure thing - I'll be at DIWD as well so we can chat about it more there. There is a great resource at http://drupal.org/patch/review which might be helpful.
Comment #21
Elijah Lynn@Greggles - Whoops, looks like I missed this comment in time for DIWD! I would have loved to take you up on your offer to talk about the patch review process. That link was very helpful though, good stuff and I will be reviewing more patches soon!
Cheers,
Elijah
Comment #22
gregglesThis module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.