WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#28516 closed defect (bug) (fixed)

DocBlock updates in wp-includes/plugin.php

Reported by: dkotter's profile dkotter Owned by: wonderboymusic's profile 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 10 years ago.
28516.2.diff​ (9.3 KB) - added by SergeyBiryukov 10 years ago.

Download all attachments as: .zip

Change History (9)

@dkotter
10 years ago

#1 @DrewAPicture
10 years ago

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

#2 @wonderboymusic
10 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.

@SergeyBiryukov
10 years ago

#3 @SergeyBiryukov
10 years ago

In 28888:

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

see #28516.

#4 @SergeyBiryukov
10 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.
10 years ago

#6 @DrewAPicture
10 years ago

In 28891:

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

See #28516.

#7 @DrewAPicture
10 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.