Closed (fixed)
Project:
Signup
Version:
4.7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
12 Dec 2006 at 21:52 UTC
Updated:
14 Apr 2007 at 07:16 UTC
Jump to comment: Most recent file
Noticed on groups.drupal.org, when an email is sent out a field is not populating with data from the database and the field name itself is showing up in the email. In this case it was the "%info" field.
Here is a copy of the email that was received:
Be reminded, ChrisB, that you signup for the following event.
The Vancouver Drupal 5 Bug Hunt!
Wed, Dec 13th, 12:00 PM
%info
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | signup_reminder_info_5x.patch.txt | 3.35 KB | dww |
| #3 | signup_reminder_info_47x.patch.txt | 3.38 KB | dww |
Comments
Comment #1
dwwthanks for the report. this doesn't utterly break the functionality of the module, it's mostly just a confusing/lame cosmetic defect, so i'm downgrading from critical to normal... i'd still like to find time to fix this in the near future, but my top priorities these days is porting project* and cvslog.module to 5.x core so that drupal.org can be upgraded. so, don't expect much from me for the next week or 2.
cheers,
-derek
Comment #2
dwwyay, i just reproduced this on a test site. it really is a bug. ;) shouldn't be too hard to fix now that i've seen it in action.
Comment #3
dwwhere's a fix for 4.7.x.
although, i noticed that if you read the fine print, the description on the reminder field doesn't mention that '%info' is a supported string substitution. ;) but, it's confusing and weird that it works in the confirmation and not in the reminder, so i'm still happy to commit this.
Comment #4
dwwand here's the 5.x version (only a minor change since the 47x one didn't apply cleanly due to t() placeholder changes in code right next to the code i had to modify).
Comment #5
dwwcommitted to HEAD, DRUPAL-5 and DRUPAL-4-7.
Comment #6
(not verified) commented