Page MenuHomePhabricator

Loginwiki fatals (TypeError: Argument 1 passed to Vector\Hooks::makeMenuItemCollapsible() must be of the type array, null given, called in /srv/mediawiki/php-1.38.0-wmf.18/skins/Vector/includes/Hooks.php on line 226)
Closed, ResolvedPublic

Description

Occurring in production.
Can be replicated at this URL:
https://login.wikimedia.org/wiki/Special:Log/block?page=User:Majavah&useskinversion=2&useskin=vector

Impact:
14 exceptions in last hour, seems to only impact login wiki logged in users
Loginwiki offline for Vector: https://login.wikimedia.org/wiki/Main_Page VS https://login.wikimedia.org/wiki/Main_Page?useskin=minerva

Event Timeline

What's special about the setup on loginwiki?
Can't replicate on beta cluster login wiki either https://login.wikimedia.beta.wmflabs.org/wiki/Special:Log/block?page=User:Majavah&useskinversion=2&useskin=vector (when anonymous)

Jdlrobson triaged this task as Unbreak Now! priority.Jan 20 2022, 5:35 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson renamed this task from TypeError: Argument 1 passed to Vector\Hooks::makeMenuItemCollapsible() must be of the type array, null given, called in /srv/mediawiki/php-1.38.0-wmf.18/skins/Vector/includes/Hooks.php on line 226 to Loginwiki fatals (TypeError: Argument 1 passed to Vector\Hooks::makeMenuItemCollapsible() must be of the type array, null given, called in /srv/mediawiki/php-1.38.0-wmf.18/skins/Vector/includes/Hooks.php on line 226).Jan 20 2022, 5:36 PM

What's special about the setup on loginwiki?

Loginwiki has most extensions disabled that normal wikis have

Can't replicate on beta cluster login wiki either https://login.wikimedia.beta.wmflabs.org/wiki/Special:Log/block?page=User:Majavah&useskinversion=2&useskin=vector

I can:

image.png (1×1 px, 407 KB)

Does loginwiki disable watchlist by any change? If so that would explain it...

I can replicate with:

$wgGroupPermissions['*']['viewmywatchlist'] = false;
``
SO I'm guessing loginwiki does this somewhere.

Change 755740 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Do not try to make watchlist collapsible on wikis where watchlist is disabled

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

Change 755740 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Do not try to make watchlist collapsible on wikis where watchlist is disabled

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

Change 755682 had a related patch set uploaded (by Majavah; author: Jdlrobson):

[mediawiki/skins/Vector@wmf/1.38.0-wmf.18] Do not try to make watchlist collapsible on wikis where watchlist is disabled

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

Change 755682 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.38.0-wmf.18] Do not try to make watchlist collapsible on wikis where watchlist is disabled

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

Mentioned in SAL (#wikimedia-operations) [2022-01-20T18:29:54Z] <taavi@deploy1002> Synchronized php-1.38.0-wmf.18/skins/Vector/includes/Hooks.php: Backport: [[gerrit:755682|Do not try to make watchlist collapsible on wikis where watchlist is disabled (T299671)]] (duration: 00m 50s)

taavi assigned this task to Jdlrobson.

Thank you for the very fast fix!