WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#28516 closed defect (bug) (fixed)

DocBlock updates in wp-includes/plugin.php

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

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 12 years ago.
28516.2.diff​ (9.3 KB ) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (9)

@dkotter
12 years ago

#1 @DrewAPicture
12 years ago

  • Component Text Changes → Plugins
  • Keywords has-patch added
  • Milestone Awaiting Review → 4.0

#2 @wonderboymusic
12 years ago

  • Owner set to wonderboymusic
  • Resolution → fixed
  • Status new → closed

In 28886:

Update @param docs in plugin.php

Props dkotter.
Fixes #28516.

@SergeyBiryukov
12 years ago

#3 @SergeyBiryukov
12 years ago

In 28888:

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

see #28516.

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

#6 @DrewAPicture
12 years ago

In 28891:

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

See #28516.

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