Hi.

I found missing translation in flag_friend.module at line 311:
'#suffix' => l('Cancel', $_GET['destination']),

must be:
'#suffix' => l(t('Cancel'), $_GET['destination']),

Please review patch.

CommentFileSizeAuthor
#2 flag_friend.module.patch718 bytesdicreat
#1 flag_friend.module.patch799 bytesdicreat
flag_friend.module.patch514 bytesdicreat

Comments

dicreat’s picture

dicreat commented
StatusFileSize
new flag_friend.module.patch799 bytes

I found another one:
$link = str_replace('Approve', 'Deny', str_replace('unflag', 'unfriend', $flag->theme('unflag', $uid)));

must be:
$link = str_replace(t('Approve'), t('Deny'), str_replace('unflag', 'unfriend', $flag->theme('unflag', $uid)));

and string:
$link = str_replace('Approve', 'Deny', $flag->theme('unflag', $uid));

must be:
$link = str_replace(t('Approve'), t('Deny'), $flag->theme('unflag', $uid));

Patch attached.

dicreat’s picture

dicreat commented
StatusFileSize
new flag_friend.module.patch718 bytes

...and one more:

-          '#options' => array(0 => 'Yes', -1 => 'No'),
+          '#options' => array(0 => t('Yes'), -1 => t('No')),
sirkitree’s picture

sirkitree commented
Status: Needs review » Fixed

Thanks! Committed http://drupal.org/cvs?commit=196258

Status: Fixed » Closed (fixed)

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