Make WordPress Core

Opened 17 months ago

Last modified 3 months ago

#62652 new defect (bug)

Ambiguous class resolution SimplePie_Core in wordpress 6.7

Reported by: vahidjavani's profile vahidjavani Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: External Libraries Keywords: needs-refresh 2nd-opinion
Focuses: Cc:

Description (last modified by sabernhardt)

I have updated WordPress to 6.7 and now getting these warnings, any idea how to fix it?

Warning: Ambiguous class resolution, "SimplePie_Core" was found in both "/.../wp-includes/SimplePie/src/Core.php" and "/.../wp-includes/SimplePie/library/SimplePie/Core.php", the first will be used.

These are the two files:
https://github.com/WordPress/WordPress/blob/6.7-branch/wp-includes/SimplePie/src/Core.php
https://github.com/WordPress/WordPress/blob/6.7-branch/wp-includes/SimplePie/library/SimplePie/Core.php

Change History (2)

#1 @sabernhardt
13 months ago

  • Component changed from General to External Libraries
  • Description modified (diff)

#2 @kitchin
3 months ago

  • Keywords needs-refresh 2nd-opinion added

WP 6.9 deleted wp-includes/SimplePie/src/Core.php.
Upgrading to WP 6.9 did not remove the file on the two installs I tested.

The deletion was in the upgrade to SimplePie 1.8.1:
https://core.trac.wordpress.org/changeset/60490
https://core.trac.wordpress.org/ticket/63717

Then it was upgraded to 1.9 before the WP 6.9 release.

As far as I can tell, 1.8.1 was not in a WP release. (You may want to search "simple pie" as well as "simplepie" in the various trac places.)

Is the lack of deletion on upgrade a bug?

Last edited 3 months ago by kitchin (previous) (diff)
Note: See TracTickets for help on using tickets.