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.

CommentFileSizeAuthor
#5 inactive_user.module_0.patch6.44 KBclaudiu.cristea

Comments

cahenesy’s picture

cahenesy commented

Crap. Let me try that again.

The e-mail subjects look like this:


[Southern Sailing Club of Lake Lanier] Account inactivity

cahenesy’s picture

cahenesy commented

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

mightyiam’s picture

mightyiam commented

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

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented

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?

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented
StatusFileSize
new inactive_user.module_0.patch6.44 KB

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

jaydub’s picture

jaydub commented
Status: Active » Needs review
deekayen’s picture

deekayen commented
Status: Needs review » Fixed

committed to DRUPAL-5 and DRUPAL-4-7

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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