WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#33652 closed defect (bug) (fixed)

@param missing in wp-includes/class-wp-user.php

Reported by: racase's profile racase Owned by: drewapicture's profile DrewAPicture
Milestone: 4.4 Priority: normal
Severity: normal Version:
Component: Users Keywords: has-patch commit
Focuses: docs Cc:

Description

@param is missing in
File: wp-includes/class-wp-user.php
Class: WP_User
Method: __set

Attachments (1)

33652.diff​ (446 bytes) - added by racase 10 years ago.

Download all attachments as: .zip

Change History (5)

@racase
10 years ago

#1 @racase
10 years ago

  • Keywords has-patch dev-feedback added

#2 @chriscct7
10 years ago

  • Keywords commit added; dev-feedback removed
  • Milestone changed from Awaiting Review to 4.4
  • Owner set to chriscct7
  • Status changed from new to accepted
  • Type changed from defect (bug) to enhancement
  • Version trunk deleted

Looks good, thanks for the report and the patch @racase

Last edited 10 years ago by chriscct7 (previous) (diff)

#3 @DrewAPicture
10 years ago

  • Component changed from Administration to Users
  • Owner changed from chriscct7 to DrewAPicture
  • Status changed from accepted to assigned
  • Type changed from enhancement to defect (bug)

Missing docs are a bug.

Looking at 33652.diff​, there are some minor formatting things with the spacing and missing periods, but I'll fix it on commit.

#4 @DrewAPicture
10 years ago

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

In 33847:

Docs: Add missing parameter documentation to the the DocBlock for WP_User::__set().

Props racase for the initial patch.
Fixes #33652.

Note: See TracTickets for help on using tickets.