Last modified: 2014-05-10 23:40:03 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T58894, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56894 - ResourceLoaderLanguageDataModule::getModifiedTime updates continuously
ResourceLoaderLanguageDataModule::getModifiedTime updates continuously
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
ResourceLoader (Other open bugs)
1.23.0
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
: performance
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-11 11:20 UTC by Niklas Laxström
Modified: 2014-05-10 23:40 UTC (History)
15 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Niklas Laxström 2013-11-11 11:20:34 UTC
+++ This bug was initially created as a clone of Bug #56856 +++

ResourceLoaderULSModule updates its modified time whenever ResourceLoaderContext::getLanguage varies. ResourceLoaderContext::getLanguage varies gets the language via the 'lang' parameter on the module request. The lang parameter is set in mediawiki.js to wgUserLanguage.

Result: the version of ext.uls.languagenames is updated continuously as users with different language settings hit the site, with the effect that practically every request for the module (and any other module fetched in the same request) is uncached.

To confirm this bug report, you can go to enwiki and check mw.loader.getVersion('ext.uls.languagenames'); . Then reload and check again.

'mediawiki.language.data' appears to likewise update continuously.
Comment 1 Bartosz Dziewoński 2013-12-04 13:45:59 UTC
Was this fixed by https://gerrit.wikimedia.org/r/#/c/99010/ ?
Comment 2 Niklas Laxström 2013-12-04 13:57:23 UTC
Joo
Comment 3 Krinkle 2014-05-10 23:40:03 UTC
Note that, while the bug creeped into the generic hash cache key making in the general ResourceLoaderModule class, the bug was in the ResourceLoaderLanguageDataModule from before it adopted the hash key method.

Ib051ef41ba239084671c30fd switched lang data module from its own homebrew logic to the general one from ResourceLoaderModule, and it can be seen in the removed lines in that diff that the previous one is:

 $key = wfMemcKey( 'resourceloader', 'langdatamodule', 'changeinfo' );

Which also doesn't fragment by context lang code or context hash.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links