[wp-trac] [WordPress Trac] #57102: Typo in MagpieRSS (deprecated)
WordPress Trac
noreply at wordpress.org
Mon Nov 14 15:45:11 UTC 2022
#57102: Typo in MagpieRSS (deprecated)
------------------------------+---------------------
Reporter: nithins53 | Owner: (none)
Type: defect (bug) | Status: new
Priority: lowest | Milestone: 6.2
Component: Feeds | Version: 6.1
Severity: trivial | Resolution:
Keywords: has-patch commit | Focuses:
------------------------------+---------------------
Changes (by desrosj):
* keywords: has-patch => has-patch commit
* priority: normal => lowest
* component: General => Feeds
* severity: normal => trivial
* milestone: Awaiting Review => 6.2
Comment:
The benefit of any updates to `rss.php` is becoming extremely low.
One thing to note is that changing the instance of
`$this->channel['descripton']` could result in unexpected behavior if
someone using the class is checking for that index. While correcting the
key's spelling would fix this bug, but would break those implementations.
Stepping back, I guess these are fine to make, but I'd honestly like to
look at potentially being able to no-op this class entirely. `MagpieRSS`
has been deprecated since WordPress 3.0, and a
[https://wpdirectory.net/search/01GHVBFXQJA6M0VH9FVT4HHF1S search of the
plugin directory shows] almost no ''real'' usage of the class (almost all
of the matches are false positives or attempts to identify crawlers, which
MagpieRSS is considered). I've opened #57107 to discuss this.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57102#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list