Missing translation in flag_friend.module at line 532:
$email['subject'] = t('!username added you as a friend !site', array('!username' => $sender->name, '!site' => 'on '. variable_get('site_name', '')));

In my case, I prefer cut off word 'on' instead translate him:
$email['subject'] = t('!username added you as a friend !site', array('!username' => $sender->name, '!site' => variable_get('site_name', '')));

Patch attached.

CommentFileSizeAuthor
#2 flag_friend.module.patch2.14 KBdicreat
#1 flag_friend.module.patch2.15 KBdicreat
flag_friend.module.patch777 bytesdicreat

Comments

dicreat’s picture

dicreat commented
StatusFileSize
new flag_friend.module.patch2.15 KB

I was found another missing strings. New full patch attached.

dicreat’s picture

dicreat commented
StatusFileSize
new flag_friend.module.patch2.14 KB

Sorry, but I don't think that new line in this case is a good decision:

        '!message' => $flag->friend_message ? t('Message:
        '). $flag->friend_message : '',

Please review new patch.

sirkitree’s picture

sirkitree commented
Status: Active » Fixed

I agree with most of your changes here, thank you.

The first part should keep the 'on', however it should be within the t().

Committed http://drupal.org/cvs?commit=223172

Status: Fixed » Closed (fixed)

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