HTML tags getting added to e-mail subjects

cahenesy - May 22, 2007 - 16:46
Project:Inactive User
Version:5.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

E-mails that the Inactive User module is sending out for my site look like this: "[Southern Sailing Club of Lake Lanier] Account inactivity]"
This is obviously ugly. :)

I have looked at the code and ad near as I can tell, it looks like these tags are getting added when the subject variable is passed into the function t(). These tags are NOT stored in the sitename field of the database.

#1

cahenesy - May 22, 2007 - 16:47

Crap. Let me try that again.

The e-mail subjects look like this:


[Southern Sailing Club of Lake Lanier] Account inactivity

#2

cahenesy - May 22, 2007 - 16:51

Last time.. with feeling (and reformatting so the bug tracker will stop rendering the tags)
The subjects look like this:

[< em >Southern Sailing Club of Lake Lanier< /em >] Account inactivity

#3

DawnLight - May 24, 2007 - 03:49

I'll wait for this to be fixed to install this module. nice work!

#4

claudiu.cristea - July 16, 2007 - 07:47

I'm getting the same... The site name (that is between "[" and "]") is surrounded by the <em> tag.

In inactive_user.module there is no place where this tag is placed. The site name is extracted with vaiable_get() function from the database...

Any help?

#5

claudiu.cristea - July 16, 2007 - 17:32

Solved! The cause of the error is in t() function. The variables with "%" prefix are parsed and surrounded by <em> tag. If you want to use the plain variable you must use the "@" character as prefix.

Attached you can find the patch to solve this bug issue.

Best regards,
Claudiu

AttachmentSize
inactive_user.module_0.patch 6.44 KB

#6

jaydub - March 20, 2008 - 04:27
Status:active» needs review

#7

deekayen - May 19, 2008 - 14:55
Status:needs review» fixed

committed to DRUPAL-5 and DRUPAL-4-7

#8

Anonymous (not verified) - June 2, 2008 - 15:04
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.