| Ryasmeen | |
| Apr 9 2019, 12:21 AM |
| F28600926: Screen Shot 2019-04-10 at 11.55.18.png | |
| Apr 10 2019, 10:56 AM |
VE is not loading for pages with music score, shows error " Uncaught Error: No class registered by that name: mwScore"
It shows the progress bar but does not complete the loading.
Example page:https://en.wikipedia.beta.wmflabs.org/wiki/User:RYasmeen_(WMF)
This is still on Beta cluster.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Release | • mmodell | T206679 1.33.0-wmf.25 deployment blockers | ||
| Resolved | Esanders | T220465 [Regression wmf.25] VE is not loading for pages with music score, shows error " Uncaught Error: No class registered by that name: mwScore" |
Did rESCRb3a898135dd7: ScoreHooks: Bundle configuration vars into JS that needs it change something in the execution order / scope / whatever, that code for score nodes isn't accessible yet when needed?
Hmmm... I'm not sure I can reproduce this on beta cluster. I don't have this problem, the VE loads to completion;
Maybe a caching issue? I'm not sure but I did a hard refresh (force refresh) to clear browser cache before testing this!
Change 502787 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/Score@master] Revert "ScoreHooks: Bundle configuration vars into JS that needs it"
Hmmm... I'm not sure I can reproduce this on beta cluster. I don't have this problem, the VE loads to completion;
Your screenshot is showing source mode, which will load fine without the score plugin. Load the visual editor on a page that has a score element on it.
According to https://www.mediawiki.org/wiki/ResourceLoader/Package_modules only the first file in a package is invoked immediately (the "main" file). VE needs all of these files to be invoked immediately. We could just make an "index.js" that invokes the other files, but doing that for every VE module going forward just means duplicating our file manifests and sending useless extra data to the client.
CC'ing @Catrope
Change 502787 merged by jenkins-bot:
[mediawiki/extensions/Score@master] Revert "ScoreHooks: Bundle configuration vars into JS that needs it"
Change 502811 had a related patch set uploaded (by Jforrester; owner: Esanders):
[mediawiki/extensions/Score@wmf/1.33.0-wmf.25] Revert "ScoreHooks: Bundle configuration vars into JS that needs it"
Change 502811 merged by jenkins-bot:
[mediawiki/extensions/Score@wmf/1.33.0-wmf.25] Revert "ScoreHooks: Bundle configuration vars into JS that needs it"
Mentioned in SAL (#wikimedia-operations) [2019-04-10T15:24:12Z] <jforrester@deploy1001> Synchronized php-1.33.0-wmf.25/extensions/Score/: UBN Revert Score changes that broke VE T220465 (duration: 01m 01s)