Closed (fixed)
Project:
Send
Version:
6.x-1.0-alpha4
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Sep 2010 at 10:09 UTC
Updated:
28 Apr 2011 at 01:51 UTC
I installed and adapted this module and so far everything is all right except one tiny little thing: I get the emails (i sent do my second mail adresse) always twice.
As I did not hack anything or installed anything extraordinary I was wondering what this bug could cause.
What went wrong? Is there any more information you need to help me?
Greetings from Germany!
Comments
Comment #1
jpitassi commentedI am having the same problem. Also when I add a message it prints the message twice on the emails.
Comment #2
bitsantos commentedI experience the same as #2. The email is sent twice and the message is printed twice in both emails.
Comment #3
bitsantos commentedI found a way to fix the double-messages. In /send/theme/send.theme.inc, lines 127-129, (6.x-1.0-alpha4) replace:
With:
Not sure how this would affect any future modules that use the Send API but it works for the Send Node module.
Comment #4
bitsantos commentedFound a workaround for the double-sending. It turns out that at some point the recipient gets doubled in the recipient array.
Just put this in at line 334 of send/includes/handlers/send.inc:
Basically makes the loop skip the current email recipient if the message has already been sent to him.
I'm sure the root cause of the double-recipients is somewhere else but in the interest of time, I'm sticking to this solution for now. :)
Comment #5
voodoo171 commentedCould you tell me something more about this solution? i tried implementing it, but i get an error :
Parse error: syntax error, unexpected T_PUBLIC in /home/users/***/sites/all/modules/send/includes/handlers/send.inc on line 338, if i replace whole section from foreach. It always makes an error with the nex "public function". I would realy appreciate some help on this. Thanks !
Comment #6
MickC commentedlike this
Comment #7
HTF commentedI also have this problem
Comment #8
dan3h commentedThanks, @bitsantos. Your fixes in #3 and #4 worked for me. I was also getting the message twice, and each message containing the body twice.
Comment #9
allie mickaCommitted a fix.
Thanks!