![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#59548 closed defect (bug) (wontfix)
Need update unused $errormsg variables on the rss.php file.
| Reported by: | upadalavipul | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Feeds | Version: | |
| Severity: | trivial | Keywords: | |
| Cc: | Focuses: | coding-standards |
Description
I have reviewed the code and found the return $errormsg variable is commenting. if it is not used then we can remove it variable and related content. if we need to use it in the future then need to related content code commenting. As of now I related code is commenting. please review the below attached patch.
Attachments (2)
Change History (5)
#2
@
3 years ago
- Keywords close added
Hi there, thanks for the ticket!
Please note that wp-includes/rss.php is a former external library (MagpieRSS), which is deprecated and may be removed in the future as per #57107.
#3
@
3 years ago
- Keywords 2nd-opinion close removed
- Milestone Awaiting Review
- Resolution → wontfix
- Severity minor → trivial
- Status new → closed
I agree to close because its value is so low, though I uploaded a patch to show the small change I suggested in case someone thinks it is worth the changeset.
Of course, the code would be much cleaner than that if functions such as fetch_rss() and _response_to_rss() simply would return false (or have some other no-op method).
#57107 may remove much more code from the deprecated MagpieRSS file.
If the no-op plan is delayed or undesired, there would be a small value in removing those 'commented out' lines with obsolete error functions:
line 414
line 426
lines 527 to 528
line 624