Improper grammar on "unable to send e-mail error message." [#279233]

Comment	File	Size	Author
#5	drupal-7-dev-279233-4.patch	641 bytes	jbomb

#2	drupal-7-dev-279233.patch	633 bytes	jbomb

Comment #1

Dutch

🇧🇪 (Ghent)

CreditAttribution: mr.baileys commented 17 March 2009 at 10:54

Version:	6.2	» 7.x-dev
Component:	install system	» base system
Issue tags:		+Quick fix, +Novice, +User interface, +Grammar

Needs to be fixed in 7.x-dev and then backported to D6.

This affects more than just the install system, since the text is in mail.inc:

    // Log errors
    if (!$message['result']) {
      watchdog('mail', 'Error sending e-mail (from %from to %to).', array('%from' => $message['from'], '%to' => $message['to']), WATCHDOG_ERROR);
      drupal_set_message(t('Unable to send e-mail. Please contact the site admin, if the problem persists.'), 'error');
    }

Log in or register to post comments

Comment #2

jbomb CreditAttribution: jbomb commented 22 March 2009 at 20:18

Assigned:

Unassigned

» jbomb

File	Size
drupal-7-dev-279233.patch	633 bytes

removed the comma as requested.

Log in or register to post comments

Comment #3

jbomb CreditAttribution: jbomb commented 22 March 2009 at 20:35

Status:

Active

» Needs review

Log in or register to post comments

Comment #4

keith.smith CreditAttribution: keith.smith commented 22 March 2009 at 21:37

I'd RTBC this, but it's very uncommon for us to abbreviate things. I realize the original string says "admin", but IMO, it should really say "administrator".