| Ciell | |
| Sep 20 2021, 5:00 PM |
| F34648898: Screenshot 2021-09-20 at 18-53-57 Central notice admin - Meta.png | |
| Sep 20 2021, 5:00 PM |
List of steps to reproduce (step by step, including full links if applicable):
What happens?:
Cannot change or set central notice banners
What should have happened instead?:
Software version (if not a Wikimedia wiki), browser information, screenshots, other information, etc:
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Ladsgroup | T280944 Phase out jQuery Migrate v3 | |||
| Resolved | BUG REPORT | None | T291410 Central notice projects and language choices not loading |
It loads briefly as plain text, then we get the blank blue boxes, so this is aparently a rendering issue somewhere.
The entries are having CSS class "ui-helper-hidden" applied. Removing this makes them visible, but clicking to add/remove them results in a javascript error:
Uncaught TypeError: Cannot read properties of undefined (reading 'prop')
at $.<computed>.<computed>._setSelected (<anonymous>:368:952)
at $.<computed>.<computed>._setSelected (load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets|jquery.plugin.jstree|jquery.ui&skin=vector&version=109wb:578)
at HTMLAnchorElement.<anonymous> (<anonymous>:371:557)
at HTMLAnchorElement.dispatch (load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets|jquery.plugin.jstree|jquery.ui&skin=vector&version=109wb:70)
at HTMLAnchorElement.elemData.handle (load.php?lang=en&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets|jquery.plugin.jstree|jquery.ui&skin=vector&version=109wb:66)Triaging as UBN as this makes it impossible to make certain changes to campaigns or create new ones.
Have to log off now, but I dug around and think this must be related to the removal of jQuery Migrate (T280944) from Meta earlier today.
@Pcoombe that looks like exactly the cause! Locally I can see the breakage with that setting false, but the multiselect works properly with wgIncludejQueryMigrate = true;
Change 722404 had a related patch set uploaded (by Legoktm; author: Legoktm):
[operations/mediawiki-config@master] Revert \"Disable jQuery Migrate on group1\"
Change 722404 merged by jenkins-bot:
[operations/mediawiki-config@master] Revert \"Disable jQuery Migrate on group1\"
Mentioned in SAL (#wikimedia-operations) [2021-09-20T19:45:58Z] <legoktm@deploy1002> Synchronized wmf-config/InitialiseSettings.php: Revert "Disable jQuery Migrate on group1" (T291410) (duration: 00m 56s)
Thanks so so much for this fix @Legoktm. CentralNotice is now working again! Thanks also @Ciell, @Pharos, @Pcoombe, @DStrine, @Ejegg :)
This should be seen as a temporary fix. This morning a Mediawiki config change was rolled out, and was just reverted for this. However we're now blocking T280944. The real fix will be to remove the unmaintained widget that CentralNotice uses for these bits of campaign config.
This has been quickly fixed on meta. So I would no longer count this as UBN since it is only blocking the removal of jQuery Migrate v3 and longer actually breaking things on production.
Change 722695 had a related patch set uploaded (by AndyRussG; author: AndyRussG):
[mediawiki/extensions/CentralNotice@master] Close all tags in multiselect for jQuery update
Change 722695 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@master] Close all tags in multiselect for jQuery update