Closed (fixed)
Project:
Comment Notify
Version:
5.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
23 Oct 2007 at 04:43 UTC
Updated:
3 Nov 2007 at 12:48 UTC
When you use a module like the smtp module to handle emails, an error occurs when sending emails, something like parameter array not set, reason, it seems, is that the drupal_mail function in the comment notify module doesn't provide a header array. I've fixed that with an empty header array.
Second thing is that the $comment was used as an array instead of an object in hook_comment, op = publish (comment is an object), this gave a fatal error. You can check this by approving multiple (or single probably) comments in the approval queue, you will get the fatal error.
| Comment | File | Size | Author |
|---|---|---|---|
| comment_notify.patch | 745 bytes | Ashraf Amayreh |
Comments
Comment #1
Christoph C. Cemper commentedthanks much for your help! looking good - I'll add you to the comitter list as discussed via email
best,christoph
Comment #2
Ashraf Amayreh commentedcommitted and release created... closing...