[wp-trac] [WordPress Trac] #57482: Remove the Unused variables in the wp-includes folder.
WordPress Trac
noreply at wordpress.org
Tue Jan 17 13:47:57 UTC 2023
#57482: Remove the Unused variables in the wp-includes folder.
--------------------------+------------------------------
Reporter: upadalavipul | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: General | Version: trunk
Severity: normal | Resolution:
Keywords: | Focuses:
--------------------------+------------------------------
Changes (by dingo_d):
* focuses: coding-standards =>
Comment:
Isn't SimplePie a bundled lib for RSS? Not sure, but maybe this should be
fixed upstream?
Also, in the Locator.php, you have just commented the lines out, if things
are not used, then it's better that they are removed (they are all
versioned files, so the change can always be reverted).
For the `$more` variable, I'm not sure it's unused (all the other feed
files have it so it could be used in some way). Also Phpstorm didn't
report it as unused for me.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57482#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list