Jdlrobson has uploaded a new change for review.

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

Change subject: Explicitly depend on mediawiki.util where needed
......................................................................

Explicitly depend on mediawiki.util where needed

Various modules use mw.util without explicitly requesting the
dependency this provides it.

This may lead to JS errors when modules are loaded in certain orders.

Bug: T138473
Change-Id: Ibded5536f49e78d6a9d8e240fb0bf7da3cac4e6d
---
M extension.json
M includes/MobileFrontend.hooks.php
2 files changed, 11 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend 
refs/changes/56/295756/1

diff --git a/extension.json b/extension.json
index dc29546..fd07cf6 100644
--- a/extension.json
+++ b/extension.json
@@ -439,6 +439,7 @@
                        ],
                        "dependencies": [
                                "mobile.modules",
+                               "mediawiki.util",
                                "mediawiki.language",
                                "mediawiki.jqueryMsg"
                        ],
@@ -603,6 +604,7 @@
                                "mobile.browser",
                                "mobile.oo",
                                "mobile.user",
+                               "mediawiki.util",
                                "mediawiki.api",
                                "mediawiki.viewport",
                                "mobile.settings",
@@ -712,6 +714,7 @@
                                "mobile.drawers",
                                "mobile.toast",
                                "mobile.messageBox",
+                               "mediawiki.util",
                                "mediawiki.confirmCloseWindow",
                                "mobile.loggingSchemas.edit"
                        ],
@@ -769,6 +772,7 @@
                                "mobile.editor.common",
                                "mobile.microAutoSize",
                                "oojs-ui.styles.icons-editing-core",
+                               "mediawiki.util",
                                "mediawiki.notification"
                        ],
                        "scripts": [
@@ -877,6 +881,7 @@
                                "desktop"
                        ],
                        "dependencies": [
+                               "mediawiki.util",
                                "mediawiki.ui.anchor",
                                "mobile.editor.common"
                        ],
@@ -1064,6 +1069,7 @@
                                "desktop"
                        ],
                        "dependencies": [
+                               "mediawiki.util",
                                "mobile.startup"
                        ],
                        "templates": {
@@ -1399,6 +1405,7 @@
                        "dependencies": [
                                "mobile.startup",
                                "mobile.toast",
+                               "mediawiki.util",
                                "mediawiki.Title"
                        ],
                        "scripts": [
@@ -1600,6 +1607,7 @@
                                "desktop"
                        ],
                        "dependencies": [
+                               "mediawiki.util",
                                "mediawiki.router",
                                "mobile.startup",
                                "mobile.mainMenu",
@@ -1658,6 +1666,7 @@
                        ],
                        "dependencies": [
                                "skins.minerva.editor",
+                               "mediawiki.util",
                                "mobile.contentOverlays"
                        ],
                        "scripts": [
@@ -1672,6 +1681,7 @@
                "skins.minerva.editor": {
                        "class": "MFResourceLoaderParsedMessageModule",
                        "dependencies": [
+                               "mediawiki.util",
                                "mediawiki.router",
                                "skins.minerva.icons.images.scripts",
                                "skins.minerva.scripts",
diff --git a/includes/MobileFrontend.hooks.php 
b/includes/MobileFrontend.hooks.php
index 05e9c63..41a63c3 100644
--- a/includes/MobileFrontend.hooks.php
+++ b/includes/MobileFrontend.hooks.php
@@ -1119,6 +1119,7 @@
                                ],
                                'mobile.notifications.overlay' => 
$resourceBoilerplate + [
                                        'dependencies' => [
+                                               'mediawiki.util',
                                                'mobile.overlays',
                                                'ext.echo.ui',
                                        ],

-- 
To view, visit https://gerrit.wikimedia.org/r/295756
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibded5536f49e78d6a9d8e240fb0bf7da3cac4e6d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to