Messages link shows error.
| Project: | Autoresponder |
| Version: | 5.x-1.0 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
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
