From 586e8fb7ec09321df726229964e8784444013fa9 Mon Sep 17 00:00:00 2001
From: Reedy <reedy@wikimedia.org>
Date: Mon, 4 Mar 2013 18:38:22 +0000
Subject: [PATCH] Revert "Commit of various live hacks"

This reverts commit 3d6851f66257d5df117688b51b7f3037c8ab6aaf.

Change-Id: I26c418d631047783ba945e6b6a7f707d876b3146
---
 includes/DefaultSettings.php             |  6 ++----
 includes/MessageBlobStore.php            |  3 ---
 includes/cache/SquidUpdate.php           | 17 +----------------
 includes/db/DatabaseMysql.php            |  4 ++--
 includes/job/jobs/HTMLCacheUpdateJob.php |  5 +----
 maintenance/tables.sql                   |  6 +++---
 6 files changed, 9 insertions(+), 32 deletions(-)

diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php
index c67cea68aa..73c097efd0 100644
--- a/includes/DefaultSettings.php
+++ b/includes/DefaultSettings.php
@@ -183,8 +183,7 @@ $wgLoadScript = false;
  * Defaults to "{$wgScriptPath}/skins".
  */
 $wgStylePath = false;
-# Broken PHP, canary mismatch -- TS
-#$wgStyleSheetPath = &$wgStylePath;
+$wgStyleSheetPath = &$wgStylePath;
 
 /**
  * The URL path of the skins directory. Should not point to an external domain.
@@ -4167,11 +4166,10 @@ $wgAutoConfirmCount = 0;
  * user who has provided an e-mail address.
  */
 $wgAutopromote = array(
-	/* test patch -- TS
 	'autoconfirmed' => array( '&',
 		array( APCOND_EDITCOUNT, &$wgAutoConfirmCount ),
 		array( APCOND_AGE, &$wgAutoConfirmAge ),
-	), */
+	),
 );
 
 /**
diff --git a/includes/MessageBlobStore.php b/includes/MessageBlobStore.php
index 6b8cf7e867..6322be7565 100644
--- a/includes/MessageBlobStore.php
+++ b/includes/MessageBlobStore.php
@@ -241,9 +241,6 @@ class MessageBlobStore {
 	}
 
 	public static function clear() {
-		// HACK: disable clear() on WMF servers
-		return;
-
 		// TODO: Give this some more thought
 		// TODO: Is TRUNCATE better?
 		try {
diff --git a/includes/cache/SquidUpdate.php b/includes/cache/SquidUpdate.php
index 10589c26be..784e30eec8 100644
--- a/includes/cache/SquidUpdate.php
+++ b/includes/cache/SquidUpdate.php
@@ -202,25 +202,10 @@ class SquidUpdate {
 					throw new MWException( "Invalid HTCP rule for URL $url\n" );
 				}
 
-				// Try and incremement value in APC cache
-				$id = apc_inc( 'squidhtcppurge' );
-				if ( $id === false ) {
-					// If false, means it didn't work
-					// Chances are that means it isn't in the cache
-					// Start saving a cached value
-					$add = apc_add( 'squidhtcppurge', 1 );
-					if ( $add === false ) {
-						wfDebugLog( 'htcp', 'Unable to set value to APC cache' );
-						$id = 0;
-					} else {
-						$id = $add;
-					}
-				}
-
 				// Construct a minimal HTCP request diagram
 				// as per RFC 2756
 				// Opcode 'CLR', no response desired, no auth
-				$htcpTransID = $id;
+				$htcpTransID = rand();
 
 				$htcpSpecifier = pack( 'na4na*na8n',
 					4, 'HEAD', strlen( $url ), $url,
diff --git a/includes/db/DatabaseMysql.php b/includes/db/DatabaseMysql.php
index 84828cb24e..fab0e96ee5 100644
--- a/includes/db/DatabaseMysql.php
+++ b/includes/db/DatabaseMysql.php
@@ -152,11 +152,11 @@ class DatabaseMysql extends DatabaseBase {
 
 		// Tell the server we're communicating with it in UTF-8.
 		// This may engage various charset conversions.
-		/*if( $wgDBmysql5 ) {
+		if( $wgDBmysql5 ) {
 			$this->query( 'SET NAMES utf8', __METHOD__ );
 		} else {
 			$this->query( 'SET NAMES binary', __METHOD__ );
-		}*/
+		}
 		// Set SQL mode, default is turning them all off, can be overridden or skipped with null
 		if ( is_string( $wgSQLMode ) ) {
 			$mode = $this->addQuotes( $wgSQLMode );
diff --git a/includes/job/jobs/HTMLCacheUpdateJob.php b/includes/job/jobs/HTMLCacheUpdateJob.php
index da17d79378..20245b3212 100644
--- a/includes/job/jobs/HTMLCacheUpdateJob.php
+++ b/includes/job/jobs/HTMLCacheUpdateJob.php
@@ -236,12 +236,9 @@ class HTMLCacheUpdateJob extends Job {
 				array( 'page_id' => $batch ) + $touchedCond,
 				__METHOD__
 			);
-			# WM patch: throttle to avoid apache CPU exhaustion -- TS
-			if ( php_sapi_name() == 'cli' ) {
-				sleep( 1 );
-			}
 		}
 
+		# Update squid
 		if ( $wgUseSquid ) {
 			$u = SquidUpdate::newFromTitles( $titleArray );
 			$u->doUpdate();
diff --git a/maintenance/tables.sql b/maintenance/tables.sql
index e501d35fe0..97d6ff20f7 100644
--- a/maintenance/tables.sql
+++ b/maintenance/tables.sql
@@ -1141,9 +1141,9 @@ CREATE TABLE /*_*/searchindex (
   si_text mediumtext NOT NULL
 ) ENGINE=MyISAM;
 
--- CREATE UNIQUE INDEX /*i*/si_page ON /*_*/searchindex (si_page);
--- CREATE FULLTEXT INDEX /*i*/si_title ON /*_*/searchindex (si_title);
--- CREATE FULLTEXT INDEX /*i*/si_text ON /*_*/searchindex (si_text);
+CREATE UNIQUE INDEX /*i*/si_page ON /*_*/searchindex (si_page);
+CREATE FULLTEXT INDEX /*i*/si_title ON /*_*/searchindex (si_title);
+CREATE FULLTEXT INDEX /*i*/si_text ON /*_*/searchindex (si_text);
 
 
 --
-- 
2.20.1