[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