WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#28516 closed defect (bug) (fixed)

DocBlock updates in wp-includes/plugin.php

Reported by: dkotter Owned by: wonderboymusic
Milestone: 4.0 Priority: normal
Severity: normal Version: 4.0
Component: Plugins Keywords: has-patch
Focuses: docs Cc:

Description

Was checking out remove_filter and happened to notice the DocBlock mentioned an extra parameter the function could accept, though the function doesn't actually accept that argument. So removed that and went ahead and cleaned up a few other DocBlocks in the same file.

Attachments (2)

28516.diff (3.2 KB) - added by dkotter 5 years ago.
28516.2.diff (9.3 KB) - added by SergeyBiryukov 5 years ago.

Download all attachments as: .zip

Change History (9)

@dkotter
5 years ago

#1 @DrewAPicture
5 years ago

  • Component changed from Text Changes to Plugins
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 4.0

#2 @wonderboymusic
5 years ago

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

In 28886:

Update @param docs in plugin.php

Props dkotter.
Fixes #28516.

#3 @SergeyBiryukov
5 years ago

In 28888:

More tweaks to @param docs in wp-includes/plugin.php.

see #28516.

#4 @SergeyBiryukov
5 years ago

In 28889:

List the expected @param type first.

see #28516.

This ticket was mentioned in IRC in #wordpress-dev by DrewAPicture. View the logs.


5 years ago

#6 @DrewAPicture
5 years ago

In 28891:

General phpDoc fixes in wp-includes/plugin.php.

See #28516.

#7 @DrewAPicture
5 years ago

I think we've about covered the spirit of this ticket. Nice work all around.

Note: See TracTickets for help on using tickets.