Page MenuHomePhabricator

Error deleting pages on MediaWiki.org: Cannot use object of type OOUI\FieldLayout as array
Closed, ResolvedPublicPRODUCTION ERROR

Description

Steps to replicate the issue (include links if applicable):

What happens?:

[a66b7206-cd80-4844-a696-27de2d6198de] 2023-05-31 04:07:07: Fatal exception of type "Error"

What should have happened instead?:
The deletion form shows up (I was trying to temporarily delete the page to clean up an old messy import that caused parallel histories). Seems to show up with all deletions of pages older than a certain point.

Error: Cannot use object of type OOUI\FieldLayout as array

from /srv/mediawiki/php-1.41.0-wmf.11/includes/htmlform/HTMLForm.php(409)
#0 /srv/mediawiki/php-1.41.0-wmf.11/includes/htmlform/HTMLForm.php(392): HTMLForm->addFields(array)
#1 /srv/mediawiki/php-1.41.0-wmf.11/includes/htmlform/OOUIHTMLForm.php(41): HTMLForm->__construct(array, RequestContext, string)
#2 /srv/mediawiki/php-1.41.0-wmf.11/includes/htmlform/HTMLForm.php(359): OOUIHTMLForm->__construct(array, RequestContext, string)
#3 /srv/mediawiki/php-1.41.0-wmf.11/includes/actions/DeleteAction.php(476): HTMLForm::factory(string, array, RequestContext)
#4 /srv/mediawiki/php-1.41.0-wmf.11/includes/actions/DeleteAction.php(352): DeleteAction->showForm(string)
#5 /srv/mediawiki/php-1.41.0-wmf.11/includes/actions/DeleteAction.php(146): DeleteAction->tempConfirmDelete()
#6 /srv/mediawiki/php-1.41.0-wmf.11/includes/actions/DeleteAction.php(116): DeleteAction->tempDelete()
#7 /srv/mediawiki/php-1.41.0-wmf.11/includes/MediaWiki.php(559): DeleteAction->show()
#8 /srv/mediawiki/php-1.41.0-wmf.11/includes/MediaWiki.php(334): MediaWiki->performAction(Article, MediaWiki\Title\Title)
#9 /srv/mediawiki/php-1.41.0-wmf.11/includes/MediaWiki.php(925): MediaWiki->performRequest()
#10 /srv/mediawiki/php-1.41.0-wmf.11/includes/MediaWiki.php(579): MediaWiki->main()
#11 /srv/mediawiki/php-1.41.0-wmf.11/index.php(50): MediaWiki->run()
#12 /srv/mediawiki/php-1.41.0-wmf.11/index.php(46): wfIndexMain()
#13 /srv/mediawiki/w/index.php(3): require(string)
#14 {main}

Details

Request URL
https://www.mediawiki.org/w/index.php?action=delete&title=Manual:PhpWiki_conversion

Event Timeline

Aklapper changed the subtype of this task from "Bug Report" to "Production Error".May 31 2023, 5:47 AM
Aklapper renamed this task from Error deleting very old pages on MediaWiki.org to Error deleting very old pages on MediaWiki.org: Cannot use object of type OOUI\FieldLayout as array.May 31 2023, 5:50 AM
Aklapper set Request URL to https://www.mediawiki.org/w/index.php?action=delete&title=Manual:PhpWiki_conversion.
Aklapper updated the task description. (Show Details)
Aklapper added a project: MediaWiki-HTMLForm.
Aklapper triaged this task as Unbreak Now! priority.May 31 2023, 5:53 AM

Change 924140 had a related patch set uploaded (by Func; author: Func):

[mediawiki/core@master] DeleteAction: Replace remaining OOUI fields

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

Func renamed this task from Error deleting very old pages on MediaWiki.org: Cannot use object of type OOUI\FieldLayout as array to Error deleting pages on MediaWiki.org: Cannot use object of type OOUI\FieldLayout as array.May 31 2023, 7:22 AM

Change 924140 merged by jenkins-bot:

[mediawiki/core@master] DeleteAction: Replace remaining OOUI fields

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

Change 924576 had a related patch set uploaded (by Daimona Eaytoy; author: Func):

[mediawiki/core@wmf/1.41.0-wmf.11] DeleteAction: Replace remaining OOUI fields

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

Change 924576 merged by jenkins-bot:

[mediawiki/core@wmf/1.41.0-wmf.11] DeleteAction: Replace remaining OOUI fields

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

Mentioned in SAL (#wikimedia-operations) [2023-05-31T13:51:31Z] <urbanecm@deploy1002> Started scap: Backport for [[gerrit:924939|Personalized praise: Fix first-ever notifications (T322452)]], [[gerrit:924576|DeleteAction: Replace remaining OOUI fields (T337809)]]

Mentioned in SAL (#wikimedia-operations) [2023-05-31T13:53:03Z] <urbanecm@deploy1002> daimona and urbanecm: Backport for [[gerrit:924939|Personalized praise: Fix first-ever notifications (T322452)]], [[gerrit:924576|DeleteAction: Replace remaining OOUI fields (T337809)]] synced to the testservers: mwdebug2002.codfw.wmnet, mwdebug2001.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug1001.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-05-31T14:02:43Z] <urbanecm@deploy1002> Finished scap: Backport for [[gerrit:924939|Personalized praise: Fix first-ever notifications (T322452)]], [[gerrit:924576|DeleteAction: Replace remaining OOUI fields (T337809)]] (duration: 11m 11s)

Func claimed this task.
matej_suchanek subscribed.

This change should be announced because it might have broken scripts / gadgets interacting with the reason input field (example).

For Tech News, please could someone summarize what information needs to be shared? I'm not sure what exact keywords to use, nor how to describe the best way for editors to fix any problems. (The example link is great, but I'm not sure if it fixes multiple (unrelated?) things, nor if it will make intuitive sense to other technical editors).
Perhaps something like this... (please rewrite it to fix inaccuracies! Or whatever will be most clear for the editors who need to read it) :

Last week, an update to [?? the API's DeleteAction hooks ??] may have broken some gadgets or user-scripts. Those can be fixed by replacing [???] with [???] (see an example fix).

More like:

Last week, an update to the delete form may have broken some gadgets or user scripts. If you need to manipulate (empty) the reason field, replace #wpReason with #wpReason > input. See an example fix.