[wp-trac] [WordPress Trac] #21521: Audit use of set_time_limit()

WordPress Trac noreply at wordpress.org
Mon Aug 19 20:27:23 UTC 2024


#21521: Audit use of set_time_limit()
-----------------------------------------------------+---------------------
 Reporter:  ryan                                     |       Owner:  (none)
     Type:  enhancement                              |      Status:  new
 Priority:  normal                                   |   Milestone:  6.7
Component:  Bootstrap/Load                           |     Version:  3.4.1
 Severity:  normal                                   |  Resolution:
 Keywords:  needs-patch dev-feedback good-first-bug  |     Focuses:  docs
-----------------------------------------------------+---------------------
Changes (by jorbin):

 * keywords:  needs-patch dev-feedback => needs-patch dev-feedback good-
     first-bug
 * focuses:   => docs
 * milestone:   => 6.7


Comment:

 Let's get this finished up for 6.7! I think the writing the remaining
 inline docs needed would be a good first bug for someone.

 Here are all of the current uses of `set_time_limit`:

 - wp-admin/includes/file.php - `wp_edit_theme_plugin_file`
 This has an inline doc and is necessary for the loopback to ensure plugin
 file updates don't cause white screens

 - wp-admin/includes/update-core.php - `update_core`
 This could use an inline doc.

 - wp-admin/includes/class-wp-automatic-updater.php -
 `WP_Automatic_Updater::update`
 This includes an inline doc about the loopback request.

 - wp-admin/includes/class-wp-upgrader.php -
 `‎WP_Upgrader::install_package`
 This could use an inline doc.

 - wp-admin/includes/ajax-actions.php - `wp_ajax_get_revision_diffs`
 This is not documented. It was introduced to fix #24757.

 - wp-includes/ID3/module.audio.mp3.php
 This is a 3rd party library. I'm ok with it as is.

 - wp-includes/PHPMailer/SMTP.php
 This is a 3rd party library. I'm ok with it as is.


 - wp-includes/comment.php - `pingback`
 This could use an inline comment explaining that this function is used to
 call external servers. It's been in WP basically forever.

 - wp-includes/class-pop3.php
 I think an inline comment could be added but it's also fine if one isn't
 since this is sort of a 3rd party code.

 - wp-includes/deprecated.php
 As this code is deprecated, I don't think it's necessary for there to be
 an inline doc.

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


More information about the wp-trac mailing list