Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#20063 closed defect (bug) (fixed)

Incorrect PHPDoc for get_current_screen

Reported by: markauk Owned by: nacin
Milestone: 3.4 Priority: normal
Severity: minor Version: 3.3.1
Component: Inline Docs Keywords:
Focuses: Cc:


The PHPDoc documentation for get_current_screen doesn't properly define the type of object which is returned. It should be:-

* @return object WP_Screen

Patch attached.

Obviously this doesn't make any difference to the running of the code, but will make life easier for anyone with IDE's which parse PHPDoc comments.

Attachments (1)

patch.diff (314 bytes) - added by markauk 4 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 follow-up: @duck_4 years ago

Since we can name the specific class it could be:

@return WP_Screen Current screen object

@markauk4 years ago

comment:2 in reply to: ↑ 1 @markauk4 years ago

Yes! I was being unnecessarily frugal with words...

Patch updated.

comment:3 @nacin4 years ago

The point was that you can omit 'object' and simply use the actual object WP_Screen as the type designator.

comment:4 @nacin4 years ago

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

In [19942]:

Use class name for @return. props markuak, fixes #20063.

comment:5 @SergeyBiryukov4 years ago

  • Milestone changed from Awaiting Review to 3.4

comment:6 @SergeyBiryukov4 years ago

  • Component changed from General to Inline Docs
Note: See TracTickets for help on using tickets.