Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#48615 closed defect (bug) (reported-upstream)

parameter types for both parameters of set_image_handler use invalid phpdoc keywords

Reported by: diddledani's profile diddledani Owned by:
Milestone: Priority: normal
Severity: trivial Version: 5.4
Component: External Libraries Keywords: has-patch close
Focuses: docs Cc:


In wp-includes/class-simplepie.php.php the param types of set_image_handler are both documented as str, which is not a valid keyword according to the phpdoc specification:

The attached patch corrects str to be string.

Attachments (1)

48615.patch (1.0 KB) - added by diddledani 5 years ago.
Patch to set param types to string

Download all attachments as: .zip

Change History (4)

5 years ago

Patch to set param types to string

#1 @SergeyBiryukov
5 years ago

  • Component changed from Feeds to External Libraries
  • Keywords close added

Thanks for the ticket and the patch! Please note that SimplePie is an external library, any changes to its files should be submitted upstream:

#2 @diddledani
5 years ago

These changes are merged into upstream version 1.5.4 (git tag 1.5.4-fixed):

#3 @SergeyBiryukov
4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to reported-upstream
  • Status changed from new to closed

Thanks for the follow-up! This should be resolved by #36669 then.

Note: See TracTickets for help on using tickets.