Page MenuHomePhabricator

ApiMobileView::getData: PoolCounter didn't return parser output
Closed, ResolvedPublic

Description

Fair amount of:

2013-08-26 18:24:03 mw1191 enwiki: [f0386546] /w/api.php?action=mobileview&page=Roman+Way+to+the+Gods&redirects=yes&prop=sections%7Ctext&sections=&sectionprop=level%7Cline&noheadings=yes&format=json Exception from line 204 of /usr/local/apache/common-local/php-1.22wmf13/extensions/MobileFrontend/includes/api/ApiMobileView.php: ApiMobileView::getData: PoolCounter didn't return parser output
#0 /usr/local/apache/common-local/php-1.22wmf13/extensions/MobileFrontend/includes/api/ApiMobileView.php(65): ApiMobileView->getData(Object(Title), false)
#1 /usr/local/apache/common-local/php-1.22wmf13/includes/api/ApiMain.php(840): ApiMobileView->execute()
#2 /usr/local/apache/common-local/php-1.22wmf13/includes/api/ApiMain.php(380): ApiMain->executeAction()
#3 /usr/local/apache/common-local/php-1.22wmf13/includes/api/ApiMain.php(351): ApiMain->executeActionWithErrorHandling()
#4 /usr/local/apache/common-local/php-1.22wmf13/api.php(73): ApiMain->execute()
#5 /usr/local/apache/common-local/w/api.php(3): require('/usr/local/apac...')
#6 {main}

Maybe $latest should be checked to see if it is not false and passed to getParserOutput().


Version: unspecified
Severity: normal

Details

Reference
bz53378

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:51 AM
bzimport set Reference to bz53378.
aaron created this task.Aug 26 2013, 10:22 PM

Hmm, but getParserOutput() already does:

if ( $oldid === null || $oldid === 0 ) {
$oldid = $this->getLatest();
}

So passing it would be a no-op?

Change 85980 had a related patch set uploaded by MaxSem:
Check latest revision in mobileview

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

Change 85980 merged by jenkins-bot:
Check latest revision in mobileview

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