use of t() creates a new string for every ecard

On line 362 of ecard.module :

drupal_set_message(t("Your ecard has been sent and a copy is stored for your reference.<br>you can view the copy <a href='".$card_copy_url."'>here</a>"));

It seems to me this should be

drupal_set_message(t("Your ecard has been sent and a copy is stored for your reference.<br>you can view the copy <a href='") . $card_copy_url . t("'>here</a>"));

The $card_copy_url should not be passed to t(). Otherwise the t() function will add a string to the table for every ecard which is sent because $card_copy_url has a different value every time, hence it is a new string.

It also makes it impossible to translate.