From: Aaron Schulz <aschulz@wikimedia.org>
Date: Fri, 18 Mar 2016 01:16:18 +0000 (-0700)
Subject: Make spreadAnyEditBlock() calls respect read-only mode
X-Git-Tag: 1.31.0-rc.0~7577^2
X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/membres/fiche.php?a=commitdiff_plain;h=5b1ae8cea0bf27fb3cddb3b5b2d1bedbe43c6d04;p=lhc%2Fweb%2Fwiklou.git

Make spreadAnyEditBlock() calls respect read-only mode

Bug: T129968
Change-Id: Ib391bd880fa9e04777a355634035183065de5b0a
---

diff --git a/includes/EditPage.php b/includes/EditPage.php
index 32687003c1..520ca57cfe 100644
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -530,11 +530,12 @@ class EditPage {
 		if ( $permErrors ) {
 			wfDebug( __METHOD__ . ": User can't edit\n" );
 			// Auto-block user's IP if the account was "hard" blocked
-			$user = $wgUser;
-			DeferredUpdates::addCallableUpdate( function() use ( $user ) {
-				$user->spreadAnyEditBlock();
-			} );
-
+			if ( !wfReadOnly() ) {
+				$user = $wgUser;
+				DeferredUpdates::addCallableUpdate( function () use ( $user ) {
+					$user->spreadAnyEditBlock();
+				} );
+			}
 			$this->displayPermissionsError( $permErrors );
 
 			return;
@@ -1735,7 +1736,9 @@ class EditPage {
 
 		if ( $wgUser->isBlockedFrom( $this->mTitle, false ) ) {
 			// Auto-block user's IP if the account was "hard" blocked
-			$wgUser->spreadAnyEditBlock();
+			if ( !wfReadOnly() ) {
+				$wgUser->spreadAnyEditBlock();
+			}
 			# Check block state against master, thus 'false'.
 			$status->setResult( false, self::AS_BLOCKED_PAGE_FOR_USER );
 			return $status;
diff --git a/includes/user/User.php b/includes/user/User.php
index 09124bfd61..831966e43c 100644
--- a/includes/user/User.php
+++ b/includes/user/User.php
@@ -3989,6 +3989,7 @@ class User implements IDBAccessObject {
 		if ( $this->isLoggedIn() && $this->isBlocked() ) {
 			return $this->spreadBlock();
 		}
+
 		return false;
 	}