Posted by sjhobson on September 8, 2009 at 6:57pm
Jump to:
| Project: | Forward |
| Version: | 6.x-1.9 |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
At the bottom of the page where the user fills out their email address and name, the form reports "You are going to email the following:" and lists a message body and subject which are not the message body and subject I filled in in the admin interface. Clearing cached files didn't help.
Comments
#1
Fixed in CVS
#2
Automatically closed -- issue fixed for 2 weeks with no activity.
#3
I'm using 6.x-1.19 and the value for "Forward Message Body: " is not being used on the forward form? This post is pretty old has the fix not made it to the current release?
#4
This appears to be because the Forward admin settings save to variables 'forward_email_subject' and 'forward_email_message' but the forward_form is referencing forward_page_subject and forward_page_message. The following code worked for me.
Index: modules/forward/forward.module
===================================================================
--- modules/forward/forward.module (revision 21118)
+++ modules/forward/forward.module (working copy)
@@ -500,7 +500,7 @@
function forward_form(&$form_state, $path = NULL, $title = NULL, $nodeapi = FALSE, $type = 'page') {
global $base_url, $user;
- $emailtype = ($path == 'epostcard') ? 'epostcard' : 'page';
+ $emailtype = ($path == 'epostcard') ? 'epostcard' : 'email';
$form = array();
$cid = array();
@@ -551,7 +551,7 @@
'#description' => t('Enter multiple addresses on separate lines or separate them with commas.'),
'#required' => TRUE,
);
- if (($emailtype == 'page') && ($nodeapi == FALSE)) {
+ if (($emailtype == 'email') && ($nodeapi == FALSE)) {
$form['message']['page'] = array(
'#type' => 'item',
'#title' => t('You are going to email the following'),