From: Kevin Israel <pleasestand@live.com>
Date: Fri, 18 Sep 2015 06:40:44 +0000 (-0400)
Subject: Database: Remove unused errorCount() method
X-Git-Tag: 1.31.0-rc.0~9974^2
X-Git-Url: https://git.heureux-cyclage.org/?a=commitdiff_plain;h=5a4c5985ae86f6b1fa296cfd6fbb9b8f447977da;p=lhc%2Fweb%2Fwiklou.git

Database: Remove unused errorCount() method

Also removed the protected $mErrorCount property used by this method.

Also removed DatabaseOracle::reportQueryError(). After removing its use
of $mErrorCount and applying relevant changes from 419221e4a71b ("Fixed
bug where catching DB errors left ignoreErrors() on"), the only effect
would be that, like for the other DB types, query errors would be
logged using wfLogDBError().

Change-Id: Ie26a5f5ef7dc5619d9db38d3eb6b7af3e6592aff
---

diff --git a/RELEASE-NOTES-1.26 b/RELEASE-NOTES-1.26
index 4463c7823e..13239de831 100644
--- a/RELEASE-NOTES-1.26
+++ b/RELEASE-NOTES-1.26
@@ -187,6 +187,7 @@ changes to languages because of Phabricator reports.
 * Watchlist tokens, SpecialResetTokens, and User::getTokenFromOption()
   are deprecated. Applications using those can work via the OAuth
   extension instead. New tokens types should not be added.
+* DatabaseBase::errorCount() was removed (unused).
 
 == Compatibility ==
 
diff --git a/includes/db/Database.php b/includes/db/Database.php
index 4d1b1c0f04..4ceebafbc5 100644
--- a/includes/db/Database.php
+++ b/includes/db/Database.php
@@ -58,7 +58,6 @@ abstract class DatabaseBase implements IDatabase {
 	protected $mSchema;
 	protected $mFlags;
 	protected $mForeign;
-	protected $mErrorCount = 0;
 	protected $mLBInfo = array();
 	protected $mDefaultBigSelects = null;
 	protected $mSchemaVars = false;
@@ -259,15 +258,6 @@ abstract class DatabaseBase implements IDatabase {
 		return $this->mTrxLevel ? $this->mTrxTimestamp : null;
 	}
 
-	/**
-	 * Get/set the number of errors logged. Only useful when errors are ignored
-	 * @param int $count The count to set, or omitted to leave it unchanged.
-	 * @return int The error count
-	 */
-	public function errorCount( $count = null ) {
-		return wfSetVar( $this->mErrorCount, $count );
-	}
-
 	/**
 	 * Get/set the table prefix.
 	 * @param string $prefix The table prefix to set, or omitted to leave it unchanged.
@@ -1111,8 +1101,6 @@ abstract class DatabaseBase implements IDatabase {
 	 * @throws DBQueryError
 	 */
 	public function reportQueryError( $error, $errno, $sql, $fname, $tempIgnore = false ) {
-		++$this->mErrorCount;
-
 		if ( $this->ignoreErrors() || $tempIgnore ) {
 			wfDebug( "SQL ERROR (ignored): $error\n" );
 		} else {
diff --git a/includes/db/DatabaseOracle.php b/includes/db/DatabaseOracle.php
index 081174af7c..87c316467d 100644
--- a/includes/db/DatabaseOracle.php
+++ b/includes/db/DatabaseOracle.php
@@ -971,20 +971,6 @@ class DatabaseOracle extends DatabaseBase {
 		return $valuedata;
 	}
 
-	function reportQueryError( $error, $errno, $sql, $fname, $tempIgnore = false ) {
-		# Ignore errors during error handling to avoid infinite
-		# recursion
-		$ignore = $this->ignoreErrors( true );
-		++$this->mErrorCount;
-
-		if ( $ignore || $tempIgnore ) {
-			wfDebug( "SQL ERROR (ignored): $error\n" );
-			$this->ignoreErrors( $ignore );
-		} else {
-			throw new DBQueryError( $this, $error, $errno, $sql, $fname );
-		}
-	}
-
 	/**
 	 * @return string Wikitext of a link to the server software's web site
 	 */