Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.For a few sites I've been working on I've made some changes to the forward module, and I'd like to know if there is a desire to include them in the official module. The changes that I've made to the forward module include:
Using Drupal's built in flood control instead of cookies.
Adding an option to have the forwarding form on same page as article using a drop down (req. ajax module).
Adding table forward_log which tracks what is sent, and click throughs.
Adding a block which lists either most recently forwarded emails, most popular today, this week, or of all time.
Changing forwarded email to html, and adding:
- article teaser
- dynamic content section, that lists recent blog posts, recent comments, new users, or can be omitted
- an additional footer message (it makes sense when you see it)
- a header graphic
I'm attaching my version of the module with a mysql file that creates a forward_log table to support the logging of forwarded messages. I'd be glad to do any coding work to get any of my changes into the module, just let me know. Also, there is a screen shot of the email over on flickr.
| Comment | File | Size | Author |
|---|---|---|---|
| forward.tar.bz | 6.37 KB | jmiccolis |
Comments
Comment #1
seanrThis looks pretty good, but is there any way to avoid requiring users to create a new mysql table? Is there any way to use the existing log table? I'd like to keep the installation as simple as possible, so users can pretty much just upload it and enable it.
Also, please take a look at the latest revision of my module in CVS (4.6 branch). I have applied a patch and fixed an injection vulnerability. If you could update yours to reflect those changes and then submit a patch, I'll be happy to commit it.
Comment #2
jmiccolis commentedI'll look into using the existing log, and your patch. If all goes well I should be able to post a version with those changes late this week.
Comment #3
seanrI have implemented the changes largely as written (though I did a lot of cleanup and created a seperate theme function). If you get a chance to look into putting it into the standard log, please let me know.
Comment #4
(not verified) commented