ErrorException from line 125 of ApiQueryQueryPage.php: PHP Notice: Undefined property: stdClass::$value
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Assigned To

Authored By

	fgiunchedi
	Jun 14 2019, 12:54 PM

Description

exception.file	       	/srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiQueryQueryPage.php:125
exception.message	       	PHP Notice: Undefined property: stdClass::$value
exception.trace	       	#0 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiQueryQueryPage.php(125): MWExceptionHandler::handleError(integer, string, string, integer, array, array)
#1 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiQueryQueryPage.php(53): ApiQueryQueryPage->run()
#2 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiQuery.php(249): ApiQueryQueryPage->execute()
#3 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiMain.php(1595): ApiQuery->execute()
#4 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiMain.php(531): ApiMain->executeAction()
#5 /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiMain.php(502): ApiMain->executeActionWithErrorHandling()
#6 /srv/mediawiki/php-1.34.0-wmf.8/api.php(87): ApiMain->execute()
#7 /srv/mediawiki/w/api.php(3): include(string)
#8 {main}
exception_id	       	XQOWfApAEDQAAHhRBggAAABM
exception_url	       	/w/api.php?action=query&list=querypage&qppage=IndexPages&format=json&qplimit=500&qpoffset=500

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Use value for sorting results in SpecialProofreadPages	mediawiki/extensions/ProofreadPage	wmf/1.34.0-wmf.10	+12 -9
	Use value for sorting results in SpecialProofreadPages	mediawiki/extensions/ProofreadPage	master	+12 -9

Customize query in gerrit

Related Objects

Mentioned In: T226429: Undefined property stdClass::$value in ApiQueryQueryPage.php
Mentioned Here: rEPRPed556868f44e: Use value for sorting results in SpecialProofreadPages
T33759: Undefined property notice in querypages API

Event Timeline

fgiunchedi created this task.Jun 14 2019, 12:54 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 14 2019, 12:54 PM

Reedy added a project: ProofreadPage.Jun 14 2019, 12:56 PM

From IRC:

See https://github.com/wikimedia/mediawiki/blob/61544d6eb2353/includes/api/ApiQueryQueryPage.php#L125
during cache look up, core QueryPage.php ensures presence of value (falling back to null, per table default) but during
raw fetch, it's up to the original query.

And for ProofreadPage, it's not selecting any value field - https://github.com/wikimedia/mediawiki-extensions-ProofreadPage/blob/fb1b06269afef0a7d424774992e729c46852fe42/includes/Special/SpecialProofreadPages.php#L211

The following might work:

Core should either detect this error and throw an exception to refuse the query if the three required fields are missing from $query['fields'], or, core should make it work without value, e.g. by adding NULL as value or some such if missing.
If solved with an exception, then ProofreadPage will still need to be fixed to actually provide a value column.

Krinkle moved this task from Untriaged to Older on the Wikimedia-production-error board.Jun 14 2019, 1:03 PM

Reedy renamed this task from ErrorException from line 125 of /srv/mediawiki/php-1.34.0-wmf.8/includes/api/ApiQueryQueryPage.php: PHP Notice: Undefined property: stdClass::$value to ErrorException from line 125 of ApiQueryQueryPage.php: PHP Notice: Undefined property: stdClass::$value.Jun 14 2019, 10:28 PM

This has produced 2M errors in the last 24h.

QueryPage was supposed to use the returned values of QueryPage::getOrderFields() to sort the results instead of value. it seems that recent code dropped partially this feature. Adding back the full support of QueryPage::getOrderFields() or, at least, strongly deprecate it seems the proper way to go.

Change 517717 had a related patch set uploaded (by Tpt; owner: Tpt):
[mediawiki/extensions/ProofreadPage@master] Use value for sorting results in SpecialProofreadPages

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

gerritbot added a project: Patch-For-Review.Jun 18 2019, 7:14 PM

Change 517782 had a related patch set uploaded (by Krinkle; owner: Tpt):
[mediawiki/extensions/ProofreadPage@wmf/1.34.0-wmf.10] Use value for sorting results in SpecialProofreadPages

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

Change 517717 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@master] Use value for sorting results in SpecialProofreadPages

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

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.11; 2019-06-26).Jun 19 2019, 3:00 AM

Krinkle mentioned this in T226429: Undefined property stdClass::$value in ApiQueryQueryPage.php.Jun 24 2019, 4:45 PM

Change 517782 merged by jenkins-bot:
[mediawiki/extensions/ProofreadPage@wmf/1.34.0-wmf.10] Use value for sorting results in SpecialProofreadPages

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

Mentioned in SAL (#wikimedia-operations) [2019-06-24T22:56:55Z] <krinkle@deploy1001> Synchronized php-1.34.0-wmf.10/extensions/ProofreadPage/includes/Special/SpecialProofreadPages.php: rEPRPed556868f44e / T225813 (duration: 00m 53s)