Last modified: 2013-12-22 09:41:58 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 T60044, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58044 - Undefined variable: noFollow in /includes/installer/LocalSettingsGenerator.php on line 368
Undefined variable: noFollow in /includes/installer/LocalSettingsGenerator.ph...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Installer (Other open bugs)
1.23.0
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-05 18:39 UTC by MWJames
Modified: 2013-12-22 09:41 UTC (History)
5 users (show)

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


Attachments

Description MWJames 2013-12-05 18:39:30 UTC
[0] produces the following stack trace [1] on Travis-CI when testing against MW master. I do wonder what happens with MW's unit tests.

## Stack
PHP Notice:  Undefined variable: noFollow in /home/travis/build/SemanticMediaWiki/phase3/includes/installer/LocalSettingsGenerator.php on line 368
PHP Stack trace:
PHP   1. {main}() /home/travis/build/SemanticMediaWiki/phase3/maintenance/install.php:0
PHP   2. require_once() /home/travis/build/SemanticMediaWiki/phase3/maintenance/install.php:133
PHP   3. CommandLineInstaller->execute() /home/travis/build/SemanticMediaWiki/phase3/maintenance/doMaintenance.php:113
PHP   4. CliInstaller->writeConfigurationFile() /home/travis/build/SemanticMediaWiki/phase3/maintenance/install.php:120
PHP   5. LocalSettingsGenerator->writeFile() /home/travis/build/SemanticMediaWiki/phase3/includes/installer/CliInstaller.php:136
PHP   6. LocalSettingsGenerator->getText() /home/travis/build/SemanticMediaWiki/phase3/includes/installer/LocalSettingsGenerator.php:163
PHP   7. LocalSettingsGenerator->getDefaultText() /home/travis/build/SemanticMediaWiki/phase3/includes/installer/LocalSettingsGenerator.php:137

[0] git clone https://gerrit.wikimedia.org/r/p/mediawiki/core.git phase3 --depth 1

[1] https://s3.amazonaws.com/archive.travis-ci.org/jobs/14997033/log.txt
Comment 1 Umherirrender 2013-12-06 14:07:30 UTC
Caused by Gerrit change #95755, because the init of the variable is inside a if and maybe not processed.
Comment 2 Gerrit Notification Bot 2013-12-06 14:52:38 UTC
Change 99654 had a related patch set uploaded by leucosticte:
Move init of $noFollow outside of if

https://gerrit.wikimedia.org/r/99654
Comment 3 Gerrit Notification Bot 2013-12-06 15:07:54 UTC
Change 99654 merged by jenkins-bot:
Move init of $noFollow outside of if

https://gerrit.wikimedia.org/r/99654
Comment 4 Umherirrender 2013-12-06 16:57:14 UTC
successfully merged

Should now be fixed on master.
Comment 5 Kevin Israel (PleaseStand) 2013-12-22 09:41:58 UTC
As far as I can tell, the change that caused this issue is not in 1.22, so the fix does not need to be backported.

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


Navigation
Links