WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#40102 closed defect (bug) (fixed)

WP_Session_Tokens::get_instance() does not have a @return inline doc

Reported by: shelob9's profile Shelob9 Owned by: drewapicture's profile DrewAPicture
Milestone: 4.9 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: docs Cc:

Description

Its return should be an instance of a subclass of WP_Session_Tokens, by default WP_User_Meta_Session_Tokens, but it has a filter, so I think WP_User_Meta_Session_Tokens|WP_Session_Tokens is best.

Attachments (1)

40102.diff​ (556 bytes) - added by Shelob9 9 years ago.

Download all attachments as: .zip

Change History (4)

@Shelob9
9 years ago

#1 @milana_cap
9 years ago

  • Keywords has-patch added

#2 @DrewAPicture
9 years ago

  • Milestone changed from Awaiting Review to 4.9

#3 @DrewAPicture
9 years ago

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

In 40998:

Docs: Add a missing return notation and to the DocBlock for WP_Session_Tokens::get_instance().

Also adds an inline reference to the session_token_manager hook in the description.

Props Shelob9 for the initial patch.
Fixes #40102.

Note: See TracTickets for help on using tickets.