Remove unused code from media.php file

[rudlinkon]

<?php // media_upload_form( $errors ); ?>

I've seen there are two places where the above code is keeping but those codes are commented so not in use.

Repository: wordpress-develop
Branch: master
Line Numbers: 2580 and 2882
File Path: /wp-admin/includes/media.php

[hellofromTonya]

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.

53764.patch​ looks good to me. Seems safe to remove both. Will mark for commit consideration.

[rudlinkon]

Replying to hellofromTonya:

Thank you so much for your kind information @hellofromTonya

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.

53764.patch​ looks good to me. Seems safe to remove both. Will mark for commit consideration.

[SergeyBiryukov]

In 51484:

Media: Remove unused code from wp-admin/includes/media.php.

Folow-up to [7043], [7062].

Props rudlinkon, hellofromTonya.
Fixes #53764.