Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#18291 closed defect (bug) (fixed)

Incorrect grammar in user notification email

Reported by: johnbillion's profile johnbillion Owned by: ryan's profile ryan
Milestone: 3.3 Priority: normal
Severity: normal Version: 3.2.1
Component: Multisite Keywords: has-patch
Focuses: Cc:

Description

When a new user is added to a site as an editor, the activation email says "You've been invited to join {site} at {url} as a editor." which is grammatically incorrect.

Not sure of the best way to fix this. A phonetic helper would be overkill, especially when it comes to i18n. Maybe we need to introduce labels for user roles (similar to labels for CPTs and taxos), or just change the wording of the email.

PS thought this had been addressed before but a Trac search brought up nothing.

Attachments (1)

18291.patch (1.3 KB) - added by SergeyBiryukov 13 years ago.

Download all attachments as: .zip

Change History (4)

#1 @nacin
13 years ago

The string is terrible, but "with the role of $%s" should be fine. At least it'll be grammatically correct.

#2 @SergeyBiryukov
13 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.3

#3 @ryan
13 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In [18487]:

Grammar fix. Props SergeyBiryukov, johnbillion. fixes #18291

Note: See TracTickets for help on using tickets.