User: Remove redundant wgClockSkewFudge code
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 15 Mar 2019 00:05:52 +0000 (00:05 +0000)
committerKrinkle <krinklemail@gmail.com>
Fri, 15 Mar 2019 20:03:54 +0000 (20:03 +0000)
commit074ce9ab1461aa0dc7bd18d65dc3c1b60061f571
treea4fb5a0c4f298d81682766911ad61539a9689b7d
parent467739271886574d47394bdb19da9c0fc772bcab
User: Remove redundant wgClockSkewFudge code

Ensuring the new value is at least as high as 1 second higher
than the current value is sufficient.

The main code paths using this are checkAndSetTouched (for user group
changes) and saveSettings(), both of which use makeUpdateConditions() which
ensures we bail out if something else already wrote to it in the mean time.
As such, there is no longer a need to make sure our time is higher than
something another server may have written, given that is no longer something
we support.

This variable was introduced in 2005 (MW 1.4) with r9403 (1d12276bcb3),
and factored out as newTouchedTimestamp() in 2007 (MW 1.8)
with r16772 (c1094ba9876).

Change-Id: I940fb0dd125286a4a348c11e2c8d197f9288a75d
RELEASE-NOTES-1.33
includes/DefaultSettings.php
includes/user/User.php