Use less fuzzy User::getDBTouched() in ApiStashEdit::getStashKey()
authorAaron Schulz <aschulz@wikimedia.org>
Thu, 21 May 2015 23:24:42 +0000 (16:24 -0700)
committerOri.livneh <ori@wikimedia.org>
Tue, 2 Jun 2015 20:31:14 +0000 (20:31 +0000)
* The getTouched() value can bump more often or vary in DCs

Change-Id: Idffa22c9dde7c7950943b104ed61f5779636b7e7

includes/User.php
includes/api/ApiStashEdit.php

index fc07b6d..828c9b1 100644 (file)
@@ -2383,7 +2383,7 @@ class User implements IDBAccessObject {
         * @return string TS_MW Timestamp
         * @since 1.26
         */
-       protected function getDBTouched() {
+       public function getDBTouched() {
                $this->load();
 
                return $this->mTouched;
index 1b27f60..1f469b8 100644 (file)
@@ -331,7 +331,7 @@ class ApiStashEdit extends ApiBase {
                        $content->getDefaultFormat(),
                        sha1( $content->serialize( $content->getDefaultFormat() ) ),
                        $user->getId() ?: md5( $user->getName() ), // account for user parser options
-                       $user->getId() ? $user->getTouched() : '-' // handle preference change races
+                       $user->getId() ? $user->getDBTouched() : '-' // handle preference change races
                ) ) );
 
                return wfMemcKey( 'prepared-edit', md5( $title->getPrefixedDBkey() ), $hash );