Posted by swmerrill on April 20, 2009 at 3:22am
Jump to:
| Project: | Autoresponder |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | cyberpunk |
| Status: | closed (fixed) |
Issue Summary
After adding more than one message to a set, you can only EDIT the first message from the Messages list page (admin/settings/autoresponder/mail/list). This is because the message set ID and the message ID are reversed in the edit setup line. The fix is to change line 758 in autoresponder.module from
l(t('Edit'), 'admin/settings/autoresponder/mail/edit/' . $mail->mset . '/' . $mail->id),
to
l(t('Edit'), 'admin/settings/autoresponder/mail/edit/' . $mail->id . '/' . $mail->mset),
Comments
#1
I haven't tested it, but presumably the same problem applies to the DELETE code on the subsequent line as well. The fix should be similar.
#2
I've experienced this bug. I think that the problem is that the edit page always gets the first mailing's content. The title appears to be ok.
#3
Fixed in dev version.