Messages link shows error.

swmerrill - July 16, 2008 - 19:21
Project:Autoresponder
Version:5.x-1.0
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Great job on the autoresponder. Thanks for doing it.

Bug in autoresponder_mail_list_form() function:

When I click on the "Messages" link to show the messages and edit them, I get an error saying that the "sid" column does not exist.

If I change the first line of code in the autoresponder_mail_list_form() function from:

$result = db_query('SELECT am.*, at.name, at.id, sid

to

$result = db_query('SELECT am.*, at.name

everything appears to work fine.

As far as I can tell, there is no sid field in either of the two tables referenced. Also, selecting at.id makes it be the $mail->id so all the lines have id's of 1/1 and only one message line shows with the description not matching the id number. Eliminating these two fields in the query removes both problems.

Also, ten lines further down, $mail->sid is referenced. It doesn't seem to do anything but since $mail->sid doesn't exist, you may want to remove it. At least, that's how it seems to me.

Thanks again for a really useful module.

Stan W. Merrill

 
 

Drupal is a registered trademark of Dries Buytaert.