Closed (fixed)
Project:
Forward
Version:
6.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Aug 2010 at 15:18 UTC
Updated:
1 Feb 2012 at 23:40 UTC
Jump to comment: Most recent file
Hello,
I changed the subject line in the settings and saved.
I forward a page via email but the original subject is used, not the new one I entered.
I tried flushing cache, running update.php etc.
Is this a known bug in 6.x-1.x-dev?
Not sure if it matters but I have the below checked in the settings.
[x] Use forward.tpl.php template instead of old style theme functions
I hope someone can help
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | forward-emailtype-886806-8.patch | 6.47 KB | osopolar |
| #6 | email_name.patch | 4.05 KB | stockliasteroid |
Comments
Comment #1
seanrThis was due to changing the email type variable from page to email and not catching the variables in the admin section. This is fixed in CVS and will be in the next release.
Comment #3
Bilmar commentedHello,
This seems to still be an issue with the latest version.
Could you kindly check what may be causing this issue?
Thank you
Comment #4
seanrAre you sure about that? Did you try 6.x-1.19? I tested it here and it worked. You will need to resave your settings page.
Comment #5
Bilmar commentedHi Sean,
I have confirmed the settings at admin/settings/forward now save with the changes properly.
However, on the forward node form for the node it shows the original text for subject and body (not the new text).
Settings with changes:
http://www.screencast.com/users/trupal218/folders/Jing/media/9a99e947-5f...
The forward form for a node:
http://www.screencast.com/users/trupal218/folders/Jing/media/88bb4ffa-a9...
Should the text for the subject and body shown on the forward node form also reflect the changes made in the settings?
Thanks for this cool module and I hope this issue may be resolved in the near future.
Regards
Comment #6
stockliasteroid commentedI verified this as well... Looks to be some inconsistencies between _email_ and _page_ variable_set/get references in the settings form and the email generation. I took a shot in the dark and assumed that it should all be _page_, as it looks like that was what things were changed to. This patch is against dev and reflects that assumption. It works for me...
Comment #7
stockliasteroid commentedSetting this to needs review...
Comment #8
osopolarAs stockliasteroid I also replaced _email_ with _page_ (I found 2 or 3 places more) and also postcard with epostcard. I also created an update hook to delete the old vars and copy one more.
Comment #9
SolomonGifford commentedThis is not fixed in drupal 7 as well. The variables are being set with "email" in the admin but pulling with "page" when showing the form.
Comment #10
john.oltman commentedI believe this was fixed in the 7.x-1.3 and 6.x-1.21 releases. If not, please reopen the issue. Thanks.