I think your PR is fine. But it needs to be tested, the best way to create a reliable times module is to create lots of tests for it.
For #2, and other questions around why things are the way they are in the code, git blame is your friend. It's very likely that it was added through some other PR that fixed some other issue, if not then you can likely just fix it. Just make sure that you add tests for it
