No time to roll a patch now, but this is a very easy fix. In email_confirm.module, t() is mistakenly used to include some HTML code:

$pending_email_notice = t('<div class="messages warning">You currently have a pending change of your e-mail address to <strong>%email</strong>', array('%email' => $user->data['email_confirm']['pending_email'])) . '</div>';

It should be this instead:
$pending_email_notice = '<div class="messages warning">' . t('You currently have a pending change of your e-mail address to <strong>%email</strong>', array('%email' => $user->data['email_confirm']['pending_email'])) . '</div>';

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

saad.waseem’s picture

Pravin Ajaaz’s picture

Here is the patch for it

Pravin Ajaaz’s picture

Status: Active » Needs review
saad.waseem’s picture

I have added a translations file for dutch you can use it if you want. You just need to import it and rest of things will work fine. And if you want to translate for other language get the translations and import them into your drupal.
The module is working fine for dustch . I have also added translations for dutch that are marked as "not translated"
in the projects own URL on Drupal.org.

saad.waseem’s picture

Status: Needs work » Needs review
greggles’s picture

Title: Translation of "You currently have a pending change of your e-mail address" is broken » Improve ability to translation email_confirm ("You currently have a pending change of your e-mail address") and mails
Priority: Minor » Normal

I wonder about the changes that remove the newlines from the emails. Looking at _user_mail_text() in core I see that there are newlines. Can you explain why you made that change?

greggles’s picture

Title: Improve ability to translation email_confirm ("You currently have a pending change of your e-mail address") and mails » Improve ability to translate email_confirm ("You currently have a pending change of your e-mail address") and mails

*ahem*

  • greggles committed 415ca7f on 7.x-1.x authored by Pravin Ajaaz
    Issue #2516686 by Pravin Ajaaz, greggles: Improve ability to translate...
greggles’s picture

Status: Needs review » Fixed

I applied the patch from #2 because I'm not sure about the changes that were added in #4.

If those are still appropriate I'd love to hear why.

Thank you both for your help to improve the module!

Status: Fixed » Closed (fixed)

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