Last modified: 2012-12-09 22:33:44 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 T35351, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 33351 - r106752 breaks parser testing
r106752 breaks parser testing
Status: RESOLVED WORKSFORME
Product: MediaWiki
Classification: Unclassified
Unit tests (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-23 19:25 UTC by OverlordQ
Modified: 2012-12-09 22:33 UTC (History)
7 users (show)

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


Attachments

Description OverlordQ 2011-12-23 19:25:24 UTC
Running test Link with double quotes in title part (literal) and alternate part (interpreted)... PHP Notice:  Undefined index: backend in /var/www/thedarkcitadel.com/w/includes/filerepo/FileRepo.php on line 45
PHP Notice:  Undefined index: backend in /var/www/thedarkcitadel.com/w/includes/filerepo/FileRepo.php on line 48
No backend defined with the name ``.
Backtrace:
#0 /var/www/thedarkcitadel.com/w/includes/filerepo/FileRepo.php(48): FileBackendGroup->get(NULL)
#1 /var/www/thedarkcitadel.com/w/includes/filerepo/RepoGroup.php(341): FileRepo->__construct(Array)
#2 /var/www/thedarkcitadel.com/w/includes/filerepo/RepoGroup.php(329): RepoGroup->newRepo(Array)
#3 /var/www/thedarkcitadel.com/w/includes/filerepo/RepoGroup.php(199): RepoGroup->initialiseRepos()
#4 /var/www/thedarkcitadel.com/w/includes/ImageFunctions.php(27): RepoGroup->checkRedirect(Object(Title))
#5 /var/www/thedarkcitadel.com/w/includes/parser/Parser.php(1890): wfIsBadImage('Denys_Savchenko...', Object(Title))
#6 /var/www/thedarkcitadel.com/w/includes/parser/Parser.php(1646): Parser->replaceInternalLinks2(false)
#7 /var/www/thedarkcitadel.com/w/includes/parser/Parser.php(1090): Parser->replaceInternalLinks('[[File:Denys Sa...')
#8 /var/www/thedarkcitadel.com/w/includes/parser/Parser.php(345): Parser->internalParse('[[File:Denys Sa...')
#9 /var/www/thedarkcitadel.com/w/tests/parser/parserTest.inc(485): Parser->parse('[[File:Denys Sa...', Object(Title), Object(ParserOptions), true, true, 1337)
#10 /var/www/thedarkcitadel.com/w/tests/parser/parserTest.inc(399): ParserTest->runTest('Link with doubl...', '[[File:Denys Sa...', '<p><a href="/in...', '', '')
#11 /var/www/thedarkcitadel.com/w/tests/parser/parserTest.inc(381): ParserTest->runTests(Object(TestFileIterator))
#12 /var/www/thedarkcitadel.com/w/tests/parserTests.php(90): ParserTest->runTestsFromFiles(Array)
#13 {main}
Comment 1 OverlordQ 2011-12-23 19:25:45 UTC
Note that this is with no special repos defined in LocalSettings.
Comment 2 Aaron Schulz 2011-12-23 19:29:20 UTC
First thing to due is fix the global leaks in parserTests.inc & UploadFromURL suite. I can remove the static setup() function in parserTests.inc and they still work...I don't know why it is there and it has no teardown counterpart. Then we can see if the problem persists.
Comment 3 OverlordQ 2011-12-23 19:44:43 UTC
For giggles, I dumped what was being passed to the construct method, when the tests begin, this is output at the beginning:

http://dpaste.com/677202/

but once it hits the failing test, the constructor is only being passed:

http://dpaste.com/677203/
Comment 4 Antoine "hashar" Musso (WMF) 2012-12-09 22:33:44 UTC
I guess this one got fixed. Either a local configuration issue or an old issue that got fixed meanwhile.  If you reproduce this in current master, feel free to reopen.

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


Navigation
Links