X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=maintenance%2FconvertUserOptions.php;h=11768c8a5618a54f7d9d5ea4838977fd9a875388;hb=23164604db9727b44d6d420b3344636da4934be1;hp=bf572448fe98aeaac00341b6a4375a46e8ea853e;hpb=ab2d63b53ed29c44e4917125555af3dc2eb9964e;p=lhc%2Fweb%2Fwiklou.git

diff --git a/maintenance/convertUserOptions.php b/maintenance/convertUserOptions.php
index bf572448fe..11768c8a56 100644
--- a/maintenance/convertUserOptions.php
+++ b/maintenance/convertUserOptions.php
@@ -26,8 +26,6 @@ require_once __DIR__ . '/Maintenance.php';
 /**
  * Maintenance script to convert user options to the new `user_properties` table.
  *
- * Do each user sequentially, since accounts can't be deleted
- *
  * @ingroup Maintenance
  */
 class ConvertUserOptions extends Maintenance {
@@ -37,12 +35,13 @@ class ConvertUserOptions extends Maintenance {
 	public function __construct() {
 		parent::__construct();
 		$this->mDescription = "Convert user options from old to new system";
+		$this->setBatchSize( 50 );
 	}
 
 	public function execute() {
 		$this->output( "...batch conversion of user_options: " );
 		$id = 0;
-		$dbw = wfGetDB( DB_MASTER );
+		$dbw = $this->getDB( DB_MASTER );
 
 		if ( !$dbw->fieldExists( 'user', 'user_options', __METHOD__ ) ) {
 			$this->output( "nothing to migrate. " );
@@ -50,14 +49,19 @@ class ConvertUserOptions extends Maintenance {
 			return;
 		}
 		while ( $id !== null ) {
-			$idCond = 'user_id > ' . $dbw->addQuotes( $id );
-			$optCond = "user_options != " . $dbw->addQuotes( '' ); // For compatibility
-			$res = $dbw->select( 'user', '*',
-				array( $optCond, $idCond ), __METHOD__,
-				array( 'LIMIT' => 50, 'FOR UPDATE' )
+			$res = $dbw->select( 'user',
+				array( 'user_id', 'user_options' ),
+				array(
+					'user_id > ' . $dbw->addQuotes( $id ),
+					"user_options != " . $dbw->addQuotes( '' ),
+				),
+				__METHOD__,
+				array(
+					'ORDER BY' => 'user_id',
+					'LIMIT' => $this->mBatchSize,
+				)
 			);
 			$id = $this->convertOptionBatch( $res, $dbw );
-			$dbw->commit( __METHOD__ );
 
 			wfWaitForSlaves();
 
@@ -77,12 +81,29 @@ class ConvertUserOptions extends Maintenance {
 		$id = null;
 		foreach ( $res as $row ) {
 			$this->mConversionCount++;
+			$insertRows = array();
+			foreach ( explode( "\n", $row->user_options ) as $s ) {
+				$m = array();
+				if ( !preg_match( "/^(.[^=]*)=(.*)$/", $s, $m ) ) {
+					continue;
+				}
 
-			$u = User::newFromRow( $row );
+				// MW < 1.16 would save even default values. Filter them out
+				// here (as in User) to avoid adding many unnecessary rows.
+				$defaultOption = User::getDefaultOption( $m[1] );
+				if ( is_null( $defaultOption ) || $m[2] != $defaultOption ) {
+					$insertRows[] = array(
+						'up_user' => $row->user_id,
+						'up_property' => $m[1],
+						'up_value' => $m[2],
+					);
+				}
+			}
 
-			$u->saveSettings();
+			if ( count( $insertRows ) ) {
+				$dbw->insert( 'user_properties', $insertRows, __METHOD__, array( 'IGNORE' ) );
+			}
 
-			// Do this here as saveSettings() doesn't set user_options to '' anymore!
 			$dbw->update(
 				'user',
 				array( 'user_options' => '' ),