Make WordPress Core

Opened 11 months ago

Closed 11 months ago

Last modified 11 months ago

#58338 closed defect (bug) (fixed)

Found a grammatical mistake in the option.php file

Reported by: nazmulhudadev's profile nazmulhudadev Owned by: audrasjb's profile audrasjb
Milestone: 6.3 Priority: low
Severity: normal Version:
Component: Options, Meta APIs Keywords: has-patch commit
Focuses: coding-standards Cc:

Description

Hi, I found a grammatical mistake in the option.php file. It's a minor issue, but I think it should be fixed.

The mistake was found in the following line:

https://github.com/WordPress/wordpress-develop/blob/trunk/src/wp-includes/option.php#L1373

Change History (17)

This ticket was mentioned in PR #4466 on WordPress/wordpress-develop by @nazmulhudadev.


11 months ago
#1

  • Keywords has-patch added

The mistake has been fixed.

Trac ticket: https://core.trac.wordpress.org/ticket/58338

#2 @audrasjb
11 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 6.3
  • Owner set to audrasjb
  • Status changed from new to accepted

Thanks for spotting this small typo. Self assigning for quick commit alongside other DocBlocks issues.

Last edited 11 months ago by audrasjb (previous) (diff)

#3 @audrasjb
11 months ago

This changeset can be committed with #58340.

@nazmulhudadev commented on PR #4466:


11 months ago
#4

Hi @mukeshpanchal27,

Thanks for reviewing the changes.

I've made the required changes in line 730 as well.

@nazmulhudadev commented on PR #4466:


11 months ago
#7

Hi @krupal-panchal,

Thanks for spotting the typo.
It has been fixed in a new PR https://github.com/WordPress/wordpress-develop/pull/4480

#8 @krupalpanchal
11 months ago

Both PR Looks good to me. Approved!

#9 @mukesh27
11 months ago

  • Keywords changes-requested added
  • Priority changed from normal to low

@nazmulhudadev Instead of multiple PR can you make single PR so it's easy to review and commit
?

@mukesh27 commented on PR #4466:


11 months ago
#10

Merge the changes in https://github.com/WordPress/wordpress-develop/pull/4480 into this and close that PR.

@nazmulhudadev commented on PR #4480:


11 months ago
#11

The changes of this PR have been merged with https://github.com/WordPress/wordpress-develop/pull/4466

#12 @nazmulhudadev
11 months ago

Hi @mukesh27,

Thanks for the suggestion. I appreciate your effort to make it easier and better.

The changes in PR #4480 have been merged into PR #4466 by a new commit.

Also, PR #4480 has been closed.

#13 @mukesh27
11 months ago

  • Keywords changes-requested removed

Awesome, Thanks @nazmulhudadev.

#14 @SergeyBiryukov
11 months ago

Thanks for the PR! The typo appears to be fixed in [55827] / #57840, sorry I didn't notice that one of the fixes in that commit was first suggested here.

We should still include the other two DocBlock edits from the PR, as they improve readability and consistency.

This ticket was mentioned in Slack in #core by chaion07. View the logs.


11 months ago

#16 @audrasjb
11 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 55884:

Docs: Fix a few more typos in Docblocks.

Follow-up to [6779], [10565], [12023], [25224], [27533], [32806], [34777], [45262], [46594], [55823], [55824], [55827].

Props nazmulhudadev, mukesh27.
Fixes #58338.
See #57840.

Note: See TracTickets for help on using tickets.