[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