&lt;p&gt;This is a one-time login for %user_name and will expire on %expiration_date.</p><p>Click on this button to login to the site and change your password.&lt;/p&gt;

should be:

&lt;p&gt;This is a one-time login for %user_name and will expire on %expiration_date.</p><p>Click on this button to login to the site and change your password.&lt;/p&gt;

The only reason this may not be a typo could be that english doesn't require a dot after a date. But I doubt it...

CommentFileSizeAuthor
d6rc_dot_typo.patch1.3 KBmeba
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

meba’s picture

meba CreditAttribution: meba commented

Oh, sorry for smashed p tags...

keith.smith’s picture

keith.smith CreditAttribution: keith.smith commented

Huh. Since we have to change this string anyway, can we take the embedded p tags out?

meba’s picture

meba CreditAttribution: meba commented

I am afraid we can't because it's too late to do it in D6, I specificaly asked Gabor about this on PM (regarding different string), but it's up to him to decide...

jvandyk’s picture

jvandyk CreditAttribution: jvandyk commented
Status: Needs review » Reviewed & tested by the community

In case it's not clear from the description (which erroneously includes the missing period in the first string) this patch inserts a period after the expiration date. We need the period to complete the sentence.

Before: will expire on %expiration_date

After: will expire on %expiration_date.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good catch. Committed to CVS HEAD. Thanks.

Anonymous’s picture

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

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