[wp-trac] [WordPress Trac] #56293: Use 'MINUTE_IN_SECONDS' for consistency in 'update.php' file

WordPress Trac noreply at wordpress.org
Mon Aug 1 05:32:30 UTC 2022


#56293: Use 'MINUTE_IN_SECONDS' for consistency in 'update.php' file
------------------------------------+-------------------------------
 Reporter:  hztyfoon                |       Owner:  audrasjb
     Type:  defect (bug)            |      Status:  reviewing
 Priority:  low                     |   Milestone:  6.1
Component:  Administration          |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:  coding-standards
------------------------------------+-------------------------------

Comment (by costdev):

 I think both make sense, but `90 * MINUTE_IN_SECONDS` is more familiar
 than `1.5 * HOUR_IN_SECONDS` - think of some sport event durations, for
 example.

 It also follows the rule of thumb that @SergeyBiryukov laid out:

 > I guess the rule of thumb would be:
 > - If the value can be represented as an integer (not a fractional)
 number of minutes (hours, etc.), use the appropriate constant multiplied
 by that number.
 > - Otherwise, keep the value as is and add a comment with the units for
 clarity.

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


More information about the wp-trac mailing list