Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#48613 closed defect (bug) (fixed)

Return type of _admin_notice_post_locked unknown phpdoc keyword

Reported by: diddledani's profile diddledani Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.4 Priority: normal
Severity: trivial Version:
Component: Administration Keywords: has-patch
Focuses: docs Cc:

Description

In wp-admin/includes/post.php the return type of _admin_notice_post_locked is documented as none, which is not a valid keyword according to the phpdoc specification: https://docs.phpdoc.org/guides/types.html

The attached patch corrects none to be void. Alternatively the @return documentation tag might be removed altogether for the same effect.

Attachments (1)

48613.patch (847 bytes) - added by diddledani 5 years ago.
Patch to set return type to void

Download all attachments as: .zip

Change History (4)

@diddledani
5 years ago

Patch to set return type to void

#1 @johnbillion
5 years ago

  • Milestone changed from Awaiting Review to 5.4
  • Version trunk deleted

#2 @SergeyBiryukov
5 years ago

I think this can just be removed. Per the documentation standards:

Note: @return void should not be used outside of the default bundled themes.

#3 @SergeyBiryukov
5 years ago

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

In 46732:

Docs: Remove incorrect @return tags.

Props diddledan.
Fixes #48613, #48614.

Note: See TracTickets for help on using tickets.