Last modified: 2013-07-29 22:32:34 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T33533, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 31533 - Check environment and display new results every time when ?page=Welcome is accessed
Check environment and display new results every time when ?page=Welcome is ac...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Installer (Other open bugs)
1.20.x
All All
: Normal major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 35125 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-10-08 17:13 UTC by Liangent
Modified: 2013-07-29 22:32 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Liangent 2011-10-08 17:13:37 UTC
When I was installing MediaWiki and reached mw-config/index.php?page=Welcome, it complained "No DB module". I went to install php5-mysql, then came back and reloaded the page. However the big red error box was still there. I tried several ways to restart the installation, including going to index.php, mw-config/index.php and clearing session cookies of the installer. No method except for the last worked but the last is really unfriendly. A newbie can be even misguided that the php-db environment is not correctly configured.
Comment 1 Liangent 2011-10-08 17:15:20 UTC
In my second attempt to reproduce the bug, I found "Restart installation" on the right. I don't know why I didn't find it at the first time but an auto-retry or a big [Retry] button which attracts more attention can be added.
Comment 2 YenTheFirst 2012-01-21 21:36:32 UTC
I can very much confirm this bug - I just spent a few hours digging through the installer to figure out why it couldn't find my MySql driver.
Comment 3 Aaron Schulz 2012-03-10 12:55:38 UTC
*** Bug 35125 has been marked as a duplicate of this bug. ***
Comment 4 MZMcBride 2012-03-10 16:46:53 UTC
This is a blocker to a 1.19 release. I've tweaked some of the bug settings accordingly. Also CC'ing Reedy.
Comment 5 Chad H. 2012-03-11 03:11:37 UTC
(In reply to comment #4)
> This is a blocker to a 1.19 release. I've tweaked some of the bug settings
> accordingly. Also CC'ing Reedy.

Why wasn't this a blocker to 1.17 or 1.18? It's not a regression.
Comment 6 MZMcBride 2012-03-11 21:06:08 UTC
(In reply to comment #5)
> (In reply to comment #4)
>> This is a blocker to a 1.19 release. I've tweaked some of the bug settings
>> accordingly. Also CC'ing Reedy.
> 
> Why wasn't this a blocker to 1.17 or 1.18? It's not a regression.

What's not a regression? It's honestly been a long time since I went through the MediaWiki installer, but you're saying that this has been the way it is for multiple versions?

There are users spending hours frustrated at a cached result. What's the virtue of caching the errors like this? And why do you think it's acceptable to release another version of MediaWiki with this issue? Compounded frustration is dangerous, especially in the installation phase. It's a quick way to get people to walk away from using MediaWiki altogether, isn't it?

Alternately, do you feel there's not an issue here and that people should just find "Restart installation" on their own? From the bug reports, it seems like there's definitely a bug here.
Comment 7 Sam Reed (reedy) 2012-03-11 21:36:06 UTC
// cache initially available databases to make sure that everything will be displayed correctly
		// after a refresh on env checks page
		$databases = $this->getVar( '_CompiledDBs-preFilter' );
		if ( !$databases ) {
			$databases = $this->getVar( '_CompiledDBs' );
			$this->setVar( '_CompiledDBs-preFilter', $databases );
		}

		$databases = array_flip ( $databases );
		foreach ( array_keys( $databases ) as $db ) {
			$installer = $this->getDBInstaller( $db );
			$status = $installer->checkPrerequisites();
			if ( !$status->isGood() ) {
				$this->showStatusMessage( $status );
			}
			if ( !$status->isOK() ) {
				unset( $databases[$db] );
			}
		}
		$databases = array_flip( $databases );
		if ( !$databases ) {
			$this->showError( 'config-no-db', $wgLang->commaList( $allNames ) );
			// @todo FIXME: This only works for the web installer!
			return false;
		}
		$this->setVar( '_CompiledDBs', $databases );



If we have already a cached list of dbs, but this count is still 0, attempt to load it again. If still nill, we could give up...
Comment 8 Gerrit Notification Bot 2013-05-25 11:25:31 UTC
Related URL: https://gerrit.wikimedia.org/r/65427 (Gerrit Change I2e7e5b4f72e7c11706d491a205e202008e2511da)
Comment 9 Gerrit Notification Bot 2013-07-29 22:30:40 UTC
Change 65427 merged by MarkAHershberger:
Installer: page refresh should refresh list of supported DBs

https://gerrit.wikimedia.org/r/65427
Comment 10 Gerrit Notification Bot 2013-07-29 22:31:18 UTC
Change 76642 had a related patch set uploaded by MarkAHershberger:
Installer: page refresh should refresh list of supported DBs

https://gerrit.wikimedia.org/r/76642
Comment 11 Gerrit Notification Bot 2013-07-29 22:32:00 UTC
Change 76642 merged by MarkAHershberger:
Installer: page refresh should refresh list of supported DBs

https://gerrit.wikimedia.org/r/76642

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links