I don't see that error, can you provide more specific reproduction steps?

Given that I wipe all gadgets on w:no:Spesial:Innstillinger#mw-prefsection-gadgets
and add the gadget "TemplateWizzard: legger til et verktøy for enkel innsetting av maler."
When I open for example w:no:Hamarkameratene for editing
Then the toolbar should have a puzzle piece to open the TemplateWizzard

What actually happen is that there are no puzzle piece, but there is an error entry in the console.

Now try to access the link w:no:Hamarkameratene for editing in debug mode, then the console error goes away and the puzzle piece appears in the toolbar.

@jeblad When you say "wipe", do you mean "disable all check boxes and save preferences", or do you mean "edit MediaWiki:Gadgets-definition and delete all gadgets"?

(The same applies to "add the gadget" – creating the gadget, or enabling the gadget.)

"disable all check boxes and save preferences", and "enabling the gadget"

But just try it, it is faster than asking me.

I suspect this may've been due to T199416 which is now fixed.

@jeblad If you see it again, do let us know. Otherwise, w'll close this by Monday.

The error isn't there now.

I had this error about 10 hours after I created this task, but it was gone at about 25 hours later. I'm pretty sure I checked it when I wrote T201686#4495861, but if I didn't then it could have been T199416. It could also have gone away because the change triggered a reload.

Not quite sure, could be the same, could also be something else. I don't want to recreate the error state on a live wiki just to check if the bug is still there. ;)

Can you try it on https://test.wikipedia.org? I've just granted you sysop there for 1 year.

Wondered why the heck I was admin there… ;)

Created the same entry as this change at MediaWiki:Gadgets-definition, which triggered the bug, and 20 minutes later it was still there while at 45 minutes later it was gone. It should although be gone when the entry was updated.

• Adding the faulty entry https://test.wikipedia.org/w/index.php?title=MediaWiki:Gadgets-definition&diff=361252&oldid=359552&diffmode=source
• Fixing the faulty entry https://test.wikipedia.org/w/index.php?title=MediaWiki:Gadgets-definition&diff=next&oldid=361252&diffmode=source

@jeblad So to clarify, the issue is not that the warning happens (given "" is an invalid dependency). The issue is that, when fixing the issue on Gadgets-definition, the change does not happen on normal pages until 1 hour later, instead of being updated within the normal 5-minute update cycle. Correct?

If so, that is indeed a bug, and sounds very related to T199416. I can confirm that the 5-minute update cycle does generally work correctly, but there might be something specific to either ResourceLoaderWikiModule or the Gadget subclass for it that is caching it for too long, or not getting purged correctly.

In short; yes.

Exact sequence to trigger the bug

Given I have added a faulty entry like * template-wizzard[ResourceLoader|rights=edit|type=general|dependencies=]|template-wizzard.js
and the JS-page exists (and so forth…)
When I fix the faulty entry like * template-wizzard[ResourceLoader|rights=edit|type=general]|template-wizzard.js
Then I expect any error to go away
and the page to load as usual

What happen is that the error goes away 20-45 minutes after the entry is corrected.

My guess; something catches the bug before the normal purge is reached.

Before

metadata[ResourceLoader|dependencies=mediawiki.util]|metadata.js|metadata.css

... ["ext.gadget.metadata","1qjy88h",[146],"site"] ... 
//> 146 correlates with "mediawiki.util"

$ mwscript eval.php --wiki test2wiki
> $gr = GadgetRepo::singleton() var_dump($gr->getGadget('metadata')->getDependencies());
array(1) {
  [0]=>
  string(14) "mediawiki.util"
} 
> echo get_class(ObjectCache::getLocalServerInstance('hash'));
APCBagOStuff

After

After editing MediaWiki:Gadgets-definition to remove [dependencies=mediawiki.util]. I find that the server-side immediately reflects the update.

$ mwscript eval.php --wiki test2wiki
> $gr = GadgetRepo::singleton(); var_dump($gr->getGadget('metadata')->getDependencies());
array(0) {
}

This shows that GadgetHooks::onPageContentSaveComplete is working as intended. It touches the WANObjectCache checkKey, and both the per-cluster cache (Memcached) and per-server cache (APC) are correctly being validated and updated or bypassed automatically.

However, over HTTP in the browser it keeps showing the stale version in the startup module. It wasn't until I manually bypassed the cache (with a random query parameter) that I got the updated result.

... ["ext.gadget.metadata","1qjy88h",[],"site"] ...

Normally I would stop here. It shows everything seems to be working fine. The caches should roll over automatically every 5 minutes for ResourceLoader, and I've tested this often enough to know that this is working fine today in production. Except when I checked back half an hour later, it was still stale in Varnish.

I tried to reproduce the same issue again, and indeed it stayed stale again. I found the culprit when looking at the headers

200 OK
...
etag: W/"1ffgsq9"
...
... ["ext.gadget.metadata","1qjy88h",[146],"site"] ...

200 OK
...
etag: W/"1ffgsq9"
...
... ["ext.gadget.metadata","1qjy88h",[],"site"] ...

It seems that while the registry contents have changed, the ETag computed by the startup module remained the unchanged. This a bug and this bug is not limited to Gadgets, and it is not limited to dependencies being set to empty.

This bug affects any change to dependencies, for any module in the system.

Change 456326 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] resourceloader: Remove selective build optimisation from getModuleContent()

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

Change 456327 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] resourceloader: Use 'enableModuleContentVersion' for startup module

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

Change 456326 merged by jenkins-bot:
[mediawiki/core@master] resourceloader: Remove selective build optimisation from getModuleContent()

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

@Krinkle Nice investigation! =)

Change 456327 merged by jenkins-bot:
[mediawiki/core@master] resourceloader: Use 'enableModuleContentVersion' for startup module

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

Change 458059 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@wmf/1.32.0-wmf.20] resourceloader: Use 'enableModuleContentVersion' for startup module

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

Change 458059 merged by jenkins-bot:
[mediawiki/core@wmf/1.32.0-wmf.20] resourceloader: Use 'enableModuleContentVersion' for startup module

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

Mentioned in SAL (#wikimedia-operations) [2018-09-05T01:42:52Z] <krinkle@deploy1001> Synchronized php-1.32.0-wmf.20/includes/resourceloader/: I8e8d3a2cd2cc - T201686 (duration: 01m 31s)