Make WordPress Core

Opened 15 years ago

Closed 15 years ago

#17770 closed defect (bug) (fixed)

Fix misspelling and minor formatting in emails

Reported by: dcowgill Owned by: nacin
Priority: normal Milestone: 3.3
Component: Mail Version: 3.2
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

To be consistent with other emails found in schema.php, upgrade.php, & ms-functions.php.

Attachments (2)

17770.diff (928 bytes ) - added by dcowgill 15 years ago.
17770.patch (534 bytes ) - added by SergeyBiryukov 15 years ago.

Download all attachments as: .zip

Change History (11)

@dcowgill
15 years ago

#1 @dcowgill
15 years ago

  • Keywords has-patch added; needs-patch removed

#2 @nacin
15 years ago

  • Milestone Awaiting Review3.2

Login is not a verb. Thanks!

What's the purpose of moving "Thanks!" up a line? Is that for consistency?

#3 @dcowgill
15 years ago

Then "Log in Here" should probably be changed to "Login Here" in /wp-admin/includes/schema.php (line 692)

"Thanks!" was moved up to match wp_new_blog_notification in /wp-admin/includes/upgrade.php (line 331)

Obviously not major stuff here - just trying to be consistent. :-)

#4 @nacin
15 years ago

I meant that Login is wrong, and the correct is Log in.

#5 @nacin
15 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In [18287]:

Login is not a verb. props dcowgill, fixes #17770.

#6 @nacin
15 years ago

In [18289]:

Login is not a verb. props dcowgill, fixes #17770.

#7 follow-up: @nacin
15 years ago

Also [18288].

I left the "Thanks" as is in wp-admin/includes/schema.php as the "Log in" was correct, no reason to break a string this late in the cycle.

#8 in reply to: ↑ 7 @SergeyBiryukov
15 years ago

  • Milestone 3.23.3
  • Resolution fixed
  • Status closedreopened

Replying to nacin:

I left the "Thanks" as is in wp-admin/includes/schema.php as the "Log in" was correct, no reason to break a string this late in the cycle.

That string and the one in ms-functions.php are exactly the same, aside from the line break and "Log in Here"/"Log in here". I guess we can merge them in 3.3. Attaching a patch.

#9 @ryan
15 years ago

  • Resolutionfixed
  • Status reopenedclosed

In [18878]:

Syncronize some strings. Props dcowgill, SergeyBiryukov. fixes #17770

Note: See TracTickets for help on using tickets.