Make WordPress Core


Ignore:
Timestamp:
09/09/2022 08:14:43 AM (3 years ago)
Author:
audrasjb
Message:

Coding Standards: Clarify time units for various timeout or expiration values.

This changeset implements a clearer and more consistent timeout/duration/expiration format. It updates time durations used in various files, as per WordPress coding standards:

  • If the value can be represented as an integer (not a fractional) number of minutes (hours, etc.), use the appropriate constant (e.g.: MINUTE_IN_SECONDS) multiplied by that number.
  • Otherwise, keep the value as is and add a comment with the units for clarity.

Follow-up to [11823], [13177], [21996], [37747], [53714].

Props hztyfoon, audrasjb, arrasel403, krupalpanchal, GaryJ, SergeyBiryukov, peterwilsoncc, rudlinkon, costdev, robinwpdeveloper.
Fixes #56293.
See #55647.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/update.php

    r53933 r54113  
    406406
    407407    if ( $doing_cron ) {
    408         $timeout = 30;
     408        $timeout = 30; // 30 seconds.
    409409    } else {
    410410        // Three seconds, plus one extra second for every 10 plugins.
     
    688688
    689689    if ( $doing_cron ) {
    690         $timeout = 30;
     690        $timeout = 30; // 30 seconds.
    691691    } else {
    692692        // Three seconds, plus one extra second for every 10 themes.
Note: See TracChangeset for help on using the changeset viewer.