Make WordPress Core

Opened 6 years ago

Closed 5 years ago

#44087 closed defect (bug) (duplicate)

Duplicate case in Entities.php

Reported by: akshayar's profile akshayar Owned by:
Milestone: Priority: lowest
Severity: normal Version:
Component: External Libraries Keywords:
Focuses: Cc:


There is duplicate 'case' present for "\x0B" in 'entity()' function of 'SimplePie_Decode_HTML_Entities' class present at 'wp-includes/SimplePie/Decode/HTML/Entities.php'

Code Reference :

Attachments (1)

44087.patch (743 bytes) - added by akshayar 6 years ago.
Removed duplicate 'case - \x0B' line

Download all attachments as: .zip

Change History (3)

6 years ago

Removed duplicate 'case - \x0B' line

#1 @swissspidy
6 years ago

  • Component changed from General to External Libraries
  • Priority changed from normal to lowest

Note that SImplePie is an external library. We don't usually write patch those in WordPress, but instead update to a newer version of a dependency or provide a patch to them, if it doesn't exist.

Looks like this has been fixed in SimplePie three years ago:

#2 @desrosj
5 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Thanks for this ticket @akshayar! As @swissspidy noted above, this was fixed upstream. Because the library needs to be updated for the fix to be included, I'm going to markt his as a duplicate of #36669, which aims to update the library in 5.4.

Note: See TracTickets for help on using tickets.