Page MenuHomePhabricator

TypeError: trigger.attr is not a function
Closed, ResolvedPublicBUG REPORT

Description

In production logs. 331 errors in the last hour. New error in 1.41.0-wmf.5 . Impacts logged in users on certain browsers.
Also appears as InvalidCharacterError: DOMTokenList.add: The token can not contain whitespace.

at https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:355:510
at Object.fire  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:287:808
at ULS.onVisible  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:125:878
at ULS.visible  <anonymous>:20:691
at ULS.show  <anonymous>:21:450
at ULS.click  <anonymous>:24:50
at HTMLButtonElement.dispatch  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:201:260
at elemData.handle  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:197:877
at Object.trigger  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:247:983
at HTMLButtonElement.<anonymous>  https://en.wikipedia.org/w/load.php?lang=en&modules=ext.echo.api%7Cext.eventLogging%2CnavigationTiming%2CwikimediaEvents%7Cext.growthExperiments.SuggestedEditSession%7Cext.uls.interface%7Cjquery%7Cjquery.makeCollapsible%7Cmediawiki.base%2Cexperiments%2Cutil%7Cmediawiki.page.watch.ajax%7Coojs-ui.styles.icons-interactions%7Cskins.vector.icons.js%7Cskins.vector.js&skin=vector-2022&version=1usc1:248:961

QA steps

  1. Login and visit https://en.wikipedia.org/wiki/Barack_Obama in Chrome
  2. Open the devtools console
  3. Click the ULS button
  4. Verify that ULS opens and that there is not an error in the console
  5. While the ULS dialog is open, scroll down the page.
  6. Verify that the sticky header appears

QA Results - Prod

ACStatusDetails
1T335148#8799694
2T335148#8799694

Event Timeline

ULS hasn't had any recent changes (that I can see) that could have triggered this: https://github.com/wikimedia/mediawiki-extensions-UniversalLanguageSelector/compare/wmf/1.41.0-wmf.4...wmf/1.41.0-wmf.5

onVisible is a callback function.

Change 910574 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@master] Fix TypeError: trigger.attr is not a function

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

Change 910574 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Fix TypeError: trigger.attr is not a function

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

Change 910499 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@wmf/1.41.0-wmf.5] Fix TypeError: trigger.attr is not a function

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

Change 910499 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.41.0-wmf.5] Fix TypeError: trigger.attr is not a function

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

Mentioned in SAL (#wikimedia-operations) [2023-04-20T20:21:17Z] <thcipriani@deploy2002> Started scap: Backport for [[gerrit:910499|Fix TypeError: trigger.attr is not a function (T335148)]]

Mentioned in SAL (#wikimedia-operations) [2023-04-20T20:22:41Z] <thcipriani@deploy2002> nray and thcipriani: Backport for [[gerrit:910499|Fix TypeError: trigger.attr is not a function (T335148)]] synced to the testservers: mwdebug2002.codfw.wmnet, mwdebug1002.eqiad.wmnet, mwdebug2001.codfw.wmnet, mwdebug1001.eqiad.wmnet

Mentioned in SAL (#wikimedia-operations) [2023-04-20T20:31:10Z] <thcipriani@deploy2002> Finished scap: Backport for [[gerrit:910499|Fix TypeError: trigger.attr is not a function (T335148)]] (duration: 09m 53s)

nray updated the task description. (Show Details)
nray subscribed.

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Ventura
Browser: Chrome
Device: MBP
Emulated Device:NA

Test Artifact(s):

QA Steps

Login and visit https://en.wikipedia.org/wiki/Barack_Obama in Chrome
Open the devtools console
Click the ULS button
✅ AC1: Verify that ULS opens and that there is not an error in the console
While the ULS dialog is open, scroll down the page.
✅ AC2: Verify that the sticky header appears

Screen Recording 2023-04-22 at 7.26.04 PM.mov.gif (890×1 px, 1 MB)