[wp-trac] [WordPress Trac] #53764: Remove unused code from media.php file

WordPress Trac noreply at wordpress.org
Fri Jul 23 17:03:43 UTC 2021


#53764: Remove unused code from media.php file
------------------------------+-------------------------------
 Reporter:  rudlinkon         |       Owner:  hellofromTonya
     Type:  defect (bug)      |      Status:  accepted
 Priority:  normal            |   Milestone:  5.9
Component:  Media             |     Version:  trunk
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |     Focuses:  coding-standards
------------------------------+-------------------------------
Changes (by hellofromTonya):

 * keywords:  has-patch => has-patch commit
 * owner:  (none) => hellofromTonya
 * status:  new => accepted


Comment:

 Hello @rudlinkon,

 Welcome to WordPress Core Trac! Thank you for this creating this ticket
 and submitting a patch.

 Both instances were introduced in ticket #5911 with changesets [7043] and
 [7062] back in 2008. No reasons shared in the ticket or code why they were
 commented out. I didn't find changesets were they were reverted and made
 active.

 [attachment:"53764.patch"] looks good to me. Seems safe to remove both.
 Will mark for `commit` consideration.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53764#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list