The exception is that 'require' is undefined.
This is because inside jQuery it is triggering queueModuleScript (line 1192) which does this when a script has loaded:
// Clear environment delete window.require; delete window.module;
however ext-language_tools.js in Ace editor is loaded separately and expects that require is still defined (having been defined in ace.js)
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Catrope | T108655 Standardise on how to access/register JavaScript interfaces | |||
| Resolved | TheDJ | T127643 [Regression wmf.15] Ace editor doesn't load in debug mode (local 'require' var ends up removed half-way) |
This also affects the SyntaxHighlight, Math and Maps plugins in VE, which all use Ace editor.
Change 276789 had a related patch set uploaded (by Jdlrobson):
Only provide global require in debug mode when needed
Change 276789 abandoned by Jdlrobson:
Only provide global require in debug mode when needed
Reason:
This doesn't work.
The ace module seems to make use of window.require quite heavily.
I'm not sure how to make this work sanely given its an external library that I know little about. Which bits
Best solution I can think of right now is RL modules could have a flag which can opt in/out of the RL require experience.
Will think about this some more.
Change 276795 had a related patch set uploaded (by Jdlrobson):
Ace should use its own require method
Change 277176 had a related patch set uploaded (by TheDJ):
Namespace define, require calls when building Ace
i've added a patch to update the Makefile to build ace with the proper options. I haven't update Ace in a while, so i'll have to run some checks if the compiled result isn't introducing other regressions.
Change 277178 had a related patch set uploaded (by TheDJ):
Update Ace and use scoped require / define
Change 277241 had a related patch set uploaded (by Esanders):
AceEditorWidget: Use namespaced 'require'
Change 277283 had a related patch set uploaded (by Jforrester):
AceEditorWidget: Use namespaced 'require'
Change 277176 merged by jenkins-bot:
Namespace define, require calls when building Ace
Change 276795 abandoned by Jdlrobson:
Ace should use its own require method
Reason:
See https://gerrit.wikimedia.org/r/#/c/277176/