Make WordPress Core

Opened 14 months ago

Last modified 11 days ago

#55604 new task (blessed)

Update SimplePie to version 1.7.0

Reported by: jrf's profile jrf Owned by:
Milestone: 6.3 Priority: normal
Severity: normal Version: 6.0
Component: External Libraries Keywords: needs-patch needs-dev-note early needs-unit-tests 2nd-opinion
Focuses: Cc:

Description

A new version of SimplePie has just been released.

This version contains a few enhancements and some bug fixes.

The most notable change, however, is that this release contains a forward-compatibility layer for the change to PSR-4 namespaced classes which is targetted for SimplePie 2.0.0.

With some similarity to Requests - the namespaced versions of the classes are in a different base directory (src) from the original versions (library).

As WP currently only includes the files in the library directory, I would like to suggest to continue doing so for now.
This still makes the forward-compatibility layer available as all files in the library directory now create a class alias to their namespaced version.

Once 2.0.0 has been released, the files included in WP, should be switched to the files from the src directory (which is currently in place mostly to allow for Composer autoloading) and should start using the namespaced names for the SimplePie classes.

I'd recommend for this update to be mentioned in a dev-note, so plugins/themes directly using SimplePie can decide for themselves when they want to change to using the namespaced names for SimplePie classes.

Refs:

I've done a cursory check of the changes and they look sane to me, but would very much like to invite a second opinion and I'd recommend testing this change (more thoroughly than usually done for upgrades like these).

I'd also like to recommend for a few cursory tests to be added to the WP test suite to ensure that both the PSR-0 as well as the PSR-4 class names load correctly when only including the library directory in WP.

I'd recommend for this update to be applied in WP 6.1 early.

Previous: #36669, #51521, #54659

Change History (23)

This ticket was mentioned in Slack in #forums by costdev. View the logs.


13 months ago

This ticket was mentioned in Slack in #meta by costdev. View the logs.


13 months ago

#3 @costdev
13 months ago

  • Keywords needs-unit-tests 2nd-opinion added

Adding 2nd-opinion and needs-unit-tests per recommendations in the ticket summary.

This ticket was mentioned in Slack in #core by costdev. View the logs.


12 months ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


12 months ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


12 months ago

This ticket was mentioned in Slack in #core by brianhenryie. View the logs.


12 months ago

This ticket was mentioned in Slack in #core by chaion07. View the logs.


11 months ago

#9 @chaion07
11 months ago

Thanks @jrf for reporting this. We reviewed this ticket during a recent bug-scrub session. Based on the feedback received from the team, we feel that this ticket needs unit testing and/ or, patches before anything can be done for this ticket to move closer to a resolution.

Props: @cu121 & @markparnell

This ticket was mentioned in Slack in #core by chaion07. View the logs.


10 months ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


10 months ago

#12 @audrasjb
10 months ago

Hello there, as per today's bug scrub, just wanted to mention that we'll give this ticket a few days before moving in to Future Release, as it is marked as early and as we're just one month away from 6.1 beta 1 :)

This ticket was mentioned in Slack in #core by chaion07. View the logs.


10 months ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


9 months ago

This ticket was mentioned in Slack in #core by chaion07. View the logs.


9 months ago

#16 @chaion07
9 months ago

  • Milestone changed from 6.1 to Future Release

Thanks @jrf for reporting this. We reviewed this ticket during a recent bug-scrub session. We are updating the ticket with the following changes:

  1. Updating milestone to Future Release considering no significant change has taken place.

Props to @audrasjb

#17 @jrf
9 months ago

  • Milestone changed from Future Release to 6.2

#18 @jrf
8 months ago

In the mean time SimplePie 1.7.0 has just been released:

Note: as the codebase has undergone a code style cleanup, the diff is kind of useless.
For a code review of the 1.7.0 release, we'll probably need to review the individual commits (with the exception of the code style one).


Based on the discussion in this ticket in the SimplePie repo, the intention is to make 1.7.0 an LTS branch which will keep supporting PHP 5.6 for the foreseeable future.

SimplePie 1.8.0 is slated to drop support for PHP < 7.2.

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


5 months ago

#20 @audrasjb
5 months ago

  • Milestone changed from 6.2 to Future Release

As per today's bug scrub: since this ticket is marked for early commit and as it is still waiting for a patch and testing, let's move it to Future Release.

#21 @hellofromTonya
3 months ago

  • Milestone changed from Future Release to 6.3

Moving this into 6.3.

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


11 days ago

#23 @audrasjb
11 days ago

  • Summary changed from Update SimplePie to version 1.6.0 to Update SimplePie to version 1.7.0
Note: See TracTickets for help on using tickets.