WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 4 weeks ago

#47955 new defect (bug)

feed-atom.php references old wp-atom.php xml:base

Reported by: tw2113 Owned by:
Milestone: 5.9 Priority: normal
Severity: normal Version:
Component: Feeds Keywords: has-patch
Focuses: Cc:

Description

wp-atom.php was a file removed, among some others, in version 3.3 but the template file for Atom based feeds was not updated to reflect.

xml:base="https://mydomain.com/wp-atom.php"

Since this base property is an optional one, we should likely have zero issue with removing it.

Attachments (1)

remove-wp-atom-php-ref.diff​ (494 bytes) - added by tw2113 2 years ago.

Download all attachments as: .zip

Change History (8)

@tw2113
2 years ago

#1 @SergeyBiryukov
2 years ago

  • Component changed from General to Feeds

#2 @tw2113
9 months ago

Just curious if we want to get this into perhaps 5.8 as well, since it's such a minuscule change and shouldn't be breaking of anything that I can think of.

#3 @SergeyBiryukov
9 months ago

  • Milestone changed from Awaiting Review to 5.8

This ticket was mentioned in ​Slack in #core by chaion07. ​View the logs.
6 months ago

#5 @JeffPaul
6 months ago

  • Keywords has-patch added

#6 @chaion07
6 months ago

  • Milestone changed from 5.8 to 5.9

Thanks to @tw2113 for reporting this. We recently reviewed this during a recent [bug-scrub session]​https://wordpress.slack.com/archives/C02RQBWTW/p1623097405364300. With Beta 1 coming up in a day punting it to 5.9. Milestone updated. Thanks

#7 @mukesh27
4 weeks ago

Patch remove-wp-atom-php-ref.diff​ working fine and ready to marge. Ping @SergeyBiryukov for review.

Note: See TracTickets for help on using tickets.