message body & subject incorrect on form user fills out [#571840]

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

Comment #1

seanr

he/him

CreditAttribution: seanr commented 5 October 2009 at 21:38

Status:

Active

» Fixed

Fixed in CVS

Comment #2

19 October 2009 at 21:40

Status:

Fixed

» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Comment #3

timdiacon CreditAttribution: timdiacon commented 14 December 2010 at 20:44

Status:

Closed (fixed)

» Active

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?

Comment #4

jonathan_hunt CreditAttribution: jonathan_hunt commented 19 December 2010 at 23:50

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'),