Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#51521 closed task (blessed) (fixed)

PHP 8.0: Update SimplePie to version 1.5.6

Reported by: jrf's profile jrf Owned by: desrosj's profile desrosj
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.6
Component: External Libraries Keywords: php8 has-patch commit
Focuses: Cc:

Description

The SimplePie libary contained a number of incompatibilites with PHP 8.0.

In as far as these have been discovered so far, these are fixed in the newest release 1.5.6, which has just come out.

With that in mind, the SimplePie library should be updated.

Refs:

Previous: #36669

Attachments (1)

51521.diff (3.8 KB) - added by desrosj 4 years ago.

Download all attachments as: .zip

Change History (7)

#1 @desrosj
4 years ago

  • Owner set to desrosj
  • Status changed from new to assigned

#2 @ravipatel
4 years ago

@desrosj
Need to improvement in code still. required changes on see tag. #51441

https://github.com/simplepie/simplepie/blob/1.5.6/library/SimplePie.php

Please review this changes which all ready approved but a third party files on not applied.

line no 2127 / 2527

* @see get_link to * @see get_link()

line no : 2991

* @see get_item_quantity to * @see get_item_quantity()

line no 2128:
* @see subscribe_url to * @see subscribe_url()

https://core.trac.wordpress.org/attachment/ticket/51441/51441-multiple-files.patch

Last edited 4 years ago by ravipatel (previous) (diff)

@desrosj
4 years ago

#3 @desrosj
4 years ago

  • Keywords has-patch added; needs-patch removed

@ravipatel Because this is an external library that is copied verbatim into WordPress, the changes you are noting would need to be made upstream in the SimplePie repository. If you'd like to see these fixed, I recommend opening an issue or a pull request there.

51521.diff applies the latest 1.5.6 update in trunk. An extra set of eyes to verify would be appreciated

#4 @ayeshrajans
4 years ago

(Extra eyes) I can also confirm that the patch applies cleanly, and the content are verbatim 1.5.6 from upstream, with changes in the class-simplepie.php file containing the class with WP's own autoloader, plus the additional phpcs:ignore PHPCompatibility.FunctionUse.ArgumentFunctionsReportCurrentValue.NeedsInspection line.

#5 @desrosj
4 years ago

  • Keywords commit added

Thanks @ayeshrajans!

#6 @desrosj
4 years ago

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

In 49176:

External Libraries: Update the SimplePie library to version 1.5.6.

This version fixes a handful of PHP 8 compatibility issues.

A full list of changes included in this update can be found on GitHub: https://github.com/simplepie/simplepie/compare/1.5.5...1.5.6.

Props jrf, ayeshrajans.
Fixes #51521.

Note: See TracTickets for help on using tickets.