Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 11 years ago

Closed 11 years ago

#33652 closed defect (bug) (fixed)

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

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

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 11 years ago.

Download all attachments as: .zip

Change History (5)

@racase
11 years ago

Attachment 33652.diff added

#1 @racase
11 years ago

Keywords has-patch dev-feedback added

#2 @chriscct7
11 years ago

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

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

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

#3 @DrewAPicture
11 years ago

Component Administration → Users
Owner changed from chriscct7 to DrewAPicture
Status accepted → assigned
Type enhancement → 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
11 years ago

Resolution → fixed
Status assigned → closed

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.

Download in other formats: