r88201 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r88200‎ | r88201 | r88202 >
Date:20:30, 15 May 2011
Author:demon
Status:ok
Tags:
Comment:
MFT r87512
Modified paths:
  • /branches/REL1_17/phase3/includes/installer/MysqlInstaller.php (modified) (history)

Diff [purge]

Index: branches/REL1_17/phase3/includes/installer/MysqlInstaller.php
@@ -419,6 +419,10 @@
420420 }
421421
422422 public function setupUser() {
 423+ $dbUser = $this->getVar( 'wgDBuser' );
 424+ if( $dbUser == $this->getVar( '_InstallUser' ) ) {
 425+ return Status::newGood();
 426+ }
423427 $status = $this->getConnection();
424428 if ( !$status->isOK() ) {
425429 return $status;
@@ -428,7 +432,6 @@
429433 $dbName = $this->getVar( 'wgDBname' );
430434 $this->db->selectDB( $dbName );
431435 $server = $this->getVar( 'wgDBserver' );
432 - $dbUser = $this->getVar( 'wgDBuser' );
433436 $password = $this->getVar( 'wgDBpassword' );
434437 $grantableNames = array();
435438
Property changes on: branches/REL1_17/phase3/includes/installer/MysqlInstaller.php
___________________________________________________________________
Added: svn:mergeinfo
436439 Merged /branches/sqlite/includes/installer/MysqlInstaller.php:r58211-58321
437440 Merged /trunk/phase3/includes/installer/MysqlInstaller.php:r82845,82847-82848,84875,84881-84882,84970,84976,85021,85066,87512
438441 Merged /branches/new-installer/phase3/includes/installer/MysqlInstaller.php:r43664-66004
439442 Merged /branches/wmf-deployment/includes/installer/MysqlInstaller.php:r53381
440443 Merged /branches/REL1_15/phase3/includes/installer/MysqlInstaller.php:r51646

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r87512Fix for r87278: if we're installing with the same user we're going to use, sk...demon21:28, 5 May 2011

Status & tagging log