Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#45032 closed enhancement (fixed)

Replace incorrect usage of `'who's` with `whose`

Reported by: burhandodhy Owned by: netweb
Priority: normal Milestone: 5.2
Component: Users Version:
Severity: minor Keywords: has-patch
Cc: Focuses: docs

Description

Related to #45026 "pluggable.php" still have the who's word

Attachments (2)

45032.patch (611 bytes ) - added by AbdulWahab610 8 years ago.
Fixed
45032.diff (553 bytes ) - added by nielsdeblaauw 8 years ago.

Download all attachments as: .zip

Change History (13)

@AbdulWahab610
8 years ago

Fixed

#1 @burhandodhy
8 years ago

  • Keywords has-patch added; needs-patch removed

#2 @netweb
8 years ago

  • Keywords needs-refresh added
  • Milestone Awaiting Review5.0

The patch needs a refresh, the current 45032.patch removes the end-of-file new line, all PHP files should have a new blank line at the end of the file

#3 follow-up: @AbdulWahab610
8 years ago

I just rechecked pluggable.php it has two newlines at the end of the file. I removed the extra end line in my patch. I can resubmit the patch without changing the endline.

#4 in reply to: ↑ 3 @netweb
8 years ago

  • Keywords commit added

Replying to AbdulWahab610:

I just rechecked pluggable.php it has two newlines at the end of the file. I removed the extra end line in my patch. I can resubmit the patch without changing the endline.

Ah, perfect, thanks @AbdulWahab610, the patch 45032.patch is fine as is then

#5 follow-up: @jeremyfelt
8 years ago

  • Keywords commit removed

The current use appears to be correct in that it's a contraction of "who is" rather than possessive.

That said, it could use some editing. :)

How about: "If $id matches the current user, there's nothing to do"

#6 @netweb
8 years ago

  • Milestone 5.05.1

Moving to the 5.1 milestone

#7 @SergeyBiryukov
8 years ago

  • Component GeneralUsers
  • Focuses docs added
  • Summary Related #45026 Replace incorrect usage of `'who's` with `whose`Replace incorrect usage of `'who's` with `whose`

#8 @netweb
8 years ago

  • Owner set to netweb
  • Status newaccepted

#9 @pento
8 years ago

  • Milestone 5.15.2

@nielsdeblaauw
8 years ago

#10 in reply to: ↑ 5 @nielsdeblaauw
8 years ago

  • Keywords needs-refresh removed

Replying to jeremyfelt:

The current use appears to be correct in that it's a contraction of "who is" rather than possessive.

That said, it could use some editing. :)

How about: "If $id matches the current user, there's nothing to do"

Applied this suggestion.

#11 @SergeyBiryukov
8 years ago

  • Resolutionfixed
  • Status acceptedclosed

In 44826:

Docs: Improve wording in a comment in wp_set_current_user().

Props nielsdeblaauw.
Fixes #45032.

Note: See TracTickets for help on using tickets.