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 | Owned by: | 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:
- https://github.com/simplepie/simplepie/releases/tag/1.5.6
- https://github.com/simplepie/simplepie/compare/1.5.5...master
Previous: #36669
Attachments (1)
Change History (7)
#3
@
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
@
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.
@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