WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#31872 closed defect (bug) (fixed)

WP_Theme's scandir() is missing @return docstring

Reported by: lamosty Owned by: DrewAPicture
Milestone: 4.3 Priority: normal
Severity: minor Version: 3.4
Component: Themes Keywords: has-patch
Focuses: docs Cc:

Description

​Looking at the scandir() WP_Theme method, it is missing a @return docstring.

Attachments (1)

wp-theme-scandir-docstring.31872.diff​ (1.5 KB) - added by lamosty 5 years ago.

Download all attachments as: .zip

Change History (9)

@lamosty
5 years ago

#1 in reply to: ↑ description @lamosty
5 years ago

I have also formatted the docstrings according to WordPress standards.

#2 @lamosty
5 years ago

  • Keywords has-patch added
  • Severity changed from normal to minor

#3 @DrewAPicture
5 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Version changed from trunk to 3.4

#4 @obenland
5 years ago

Docs change only, @DrewAPicture I assume you'll push that forward?

#5 @DrewAPicture
5 years ago

  • Owner set to DrewAPicture
  • Status changed from new to reviewing

#6 @DrewAPicture
5 years ago

In 32344:

Properly document WP_Theme::scandir() as a static method, adjust parameter documentation for formatting and clarity.

See #31872.

#7 @DrewAPicture
5 years ago

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

In 32345:

Add a missing @return tag and description to the DocBlock for WP_Theme::scandir().

Props lamosty.
Fixes #31872.

#8 @DrewAPicture
5 years ago

  • Milestone changed from Future Release to 4.3
Note: See TracTickets for help on using tickets.