From 3296605ef2272e3ab4fdb5c5636479bb4da26d8b Mon Sep 17 00:00:00 2001
From: Sam Reed <reedy@users.mediawiki.org>
Date: Tue, 7 Jun 2011 21:28:47 +0000
Subject: [PATCH] Revert r87292

---
 includes/User.php                       | 12 +++++-------
 includes/specials/SpecialUserlogin.php  |  2 +-
 includes/specials/SpecialUserrights.php |  2 +-
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/includes/User.php b/includes/User.php
index 0ce2866655..1ebc3a4a4b 100644
--- a/includes/User.php
+++ b/includes/User.php
@@ -1743,7 +1743,7 @@ class User {
 			$wgMemc->set( $key, $val ? 1 : 0, 1800 );
 		}
 		if ( $changed ) {
-			$this->invalidateCache( true );
+			$this->invalidateCache();
 		}
 	}
 
@@ -1776,15 +1776,13 @@ class User {
 	 * Immediately touch the user data cache for this account.
 	 * Updates user_touched field, and removes account data from memcached
 	 * for reload on the next hit.
-	 *
-	 * @param $doDatabaseUpdate bool Do you really need to update the database? Really?
 	 */
-	function invalidateCache( $doDatabaseUpdate = false ) {
+	function invalidateCache() {
 		if( wfReadOnly() ) {
 			return;
 		}
 		$this->load();
-		if( $this->mId && $doDatabaseUpdate ) {
+		if( $this->mId ) {
 			$this->mTouched = self::newTouchedTimestamp();
 
 			$dbw = wfGetDB( DB_MASTER );
@@ -2247,7 +2245,7 @@ class User {
 		$this->mGroups[] = $group;
 		$this->mRights = User::getGroupPermissions( $this->getEffectiveGroups( true ) );
 
-		$this->invalidateCache( true );
+		$this->invalidateCache();
 	}
 
 	/**
@@ -2269,7 +2267,7 @@ class User {
 		$this->mGroups = array_diff( $this->mGroups, array( $group ) );
 		$this->mRights = User::getGroupPermissions( $this->getEffectiveGroups( true ) );
 
-		$this->invalidateCache( true );
+		$this->invalidateCache();
 	}
 
 	/**
diff --git a/includes/specials/SpecialUserlogin.php b/includes/specials/SpecialUserlogin.php
index cd22191dca..a96205af18 100644
--- a/includes/specials/SpecialUserlogin.php
+++ b/includes/specials/SpecialUserlogin.php
@@ -658,7 +658,7 @@ class LoginForm extends SpecialPage {
 					$wgUser->setOption( 'rememberpassword', $this->mRemember ? 1 : 0 );
 					$wgUser->saveSettings();
 				} else {
-					$wgUser->invalidateCache( true );
+					$wgUser->invalidateCache();
 				}
 				$wgUser->setCookies();
 				self::clearLoginToken();
diff --git a/includes/specials/SpecialUserrights.php b/includes/specials/SpecialUserrights.php
index b4381dad1e..41b1fb9d2b 100644
--- a/includes/specials/SpecialUserrights.php
+++ b/includes/specials/SpecialUserrights.php
@@ -231,7 +231,7 @@ class UserrightsPage extends SpecialPage {
 		$newGroups = array_unique( $newGroups );
 
 		// Ensure that caches are cleared
-		$user->invalidateCache( true );
+		$user->invalidateCache();
 
 		wfDebug( 'oldGroups: ' . print_r( $oldGroups, true ) );
 		wfDebug( 'newGroups: ' . print_r( $newGroups, true ) );
-- 
2.20.1