Closed (fixed)
Project:
Comment Notify
Version:
5.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Oct 2008 at 16:42 UTC
Updated:
10 Mar 2010 at 22:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
gregglesI'll have to investigate this issue. It may be related to the way that the publish operation of hook_comment was implemented in Drupal 6.x.
Have you tried it without comment moderation? Can you confirm it works in that situation?
Comment #2
gregglesI just tried this and it worked fine for me. Note that comment_notify will leave a message in the watchdog for every e-mail it sends, so you can figure out if it is a comment_notify issue (nothing in watchdog) or a mail server issue (message in watchdog, no mail).
Also, comment notify doesn't send messages to the comment author for their comments.
Comment #3
bejayoharen commentedYes, it works fine for unmoderated users, so I don't think it's a mailserver issue. I tried with a variety of emails and users. I will try again and check the logs. FYI: I did not know what you meant by watchdog b/c it is no longer called that in Drupal 6.
Comment #4
bejayoharen commentedAnon users on my system get their comments moderated, so here's what I did:
- made an anon comment. (asked to be notified)
- let it through.
- made another anon comment with another email. (also asked to be notfied)
- let it through.
- checked logs. Nothing. and never received emails. (admin/reports/dblog)
- made a comment as a privileged user. (again asked to be notified, but I guess that's neither here nor there)
- checked logs. it said it sent two emails, and by that time I had had two emails in my inbox, one to each email addy I had for each previous comment.
Thanks for your help so far.
Comment #5
gregglesSorry, I forgot that watchdog is now dblog ;)
One thing I would suggest is to use the 6.x-1.x-dev version. 6.x-1.1 has a lot of serious holes and I am about to make a 6.x-1.2 from the 6.x-1.x-dev.
I will try to run through the scenario you describe.
Comment #6
bejayoharen commentedSeems to be doing the same thing, I'm afraid.
Comment #7
lupo commentedHi.
I am experiencing this same issue. Moderated comments, no emails about posted comments.
I just thought I should mention it as the module description does ask to have problems noted here so they can be worked on.
Thank you for your efforts with this module. I appreciate it very much.
Dave
Comment #8
gemini commentedSince I posted a duplicate issue, I'm subscribing here.
Comment #9
akael commentedI am having the same issue, has this been resolved yet?
Comment #10
gregglesI did some more investigation into hook_comment and I'm not sure of the best way to solve it.
Comment Notify relies on hook_comment being called with "publish" as the operation. That works fine in some cases, but not when a comment is moved from unpublished to published. Frankly, there is a lot of inconsistency in the way that hook_comment is called and when the different operations are or are not used.
Would it be impossible to use unmoderated comments on your sites? Comment notify should work fine with Captch/ReCaptch/Mollom based solutions to prevent spam.
Comment #11
bejayoharen commentedWe expect to have concerns other than spam on our site: preserving anonymity, fostering a community spirit, etc.
Comment #12
gregglesThis is likely to be fixed by #334735: Remove patching core dependency. I'm tempted to mark this as a duplicate.
Note that #334735: Remove patching core dependency is dependent on #186025: Distinguish between new comments and comment edits.
Comment #13
gregglesI've now fixed #334735: Remove patching core dependency for 6.x. So, I think this is fixed. When a new tarball of 6.x-1.x-dev is created in a few hours, please help test it out. I plan to create a new release of the module in the next few days.
Comment #14
bejayoharen commentedno, still not working.
Comment #15
gregglesHmmm - ok.
Comment #16
gregglesOk, here's a patch that I believe solves this issue.
Comment #17
gregglesI've now committed this to the 6.x branch http://drupal.org/cvs?commit=180162
Comment #18
gregglesI forgot to change the Version to 5.x.
If it's not working for 6.x please re-open and move back to 6.x.
Comment #19
gsgchoi commentedI am using drupal 6.x and it doesn't send any email if I use comment approval queue. Is this bug already known as fixed? Please let me know about the latest update! However when I tried it, it didn't work.
Comment #20
gsgchoi commentedI am storing anonymous comments in approval queue and this doesn't send any email once it publishes new comments. Is this known as fixed? Please let me know about the latest status.
I checked dblog and I don't see any email sending action thus I assume that this is not a mail server problem.
Comment #21
gsgchoi commentedFYI, I am using Drupal 6.x
Comment #22
otavio commentedOn the same boat here.
Installed 5.x-2.x-dev, patched the core comment module but can only get notification if moderation is off.
Is this working i D6?
Comment #23
gregglesI believe it is working in d6 - I tested it a while ago when I committed the patch, but have not tested recently.
Comment #24
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.