Page MenuHomePhabricator

Implement a basic `phutil_units()` function for writing "30 days in seconds" readably
ClosedPublic

Authored by epriestley on Feb 1 2014, 11:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 28, 2:58 PM
Unknown Object (File)
Mon, Jan 27, 1:34 PM
Unknown Object (File)
Mon, Jan 27, 5:21 AM
Unknown Object (File)
Fri, Jan 24, 4:20 PM
Unknown Object (File)
Tue, Jan 21, 11:34 AM
Unknown Object (File)
Dec 28 2024, 11:43 PM
Unknown Object (File)
Dec 26 2024, 2:50 PM
Unknown Object (File)
Dec 17 2024, 8:26 AM
Subscribers

Details

Summary

Fixes T4325. We have a fair amount of (60 * 60 * 24 * 15) // 15 days stuff in the codebase, and at least one of them was wrong (15 hours, when 15 minutes was probably intended). I was about to write some more, so I wrote this instead.

Make these self-documenting and harder to get wrong.

(This function could be more sophisticated, but I think this is the only real use case we hit.)

Test Plan

Added and ran unit tests.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped