Last modified: 2014-09-02 09:10:31 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 T71279, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69279 - Sitenotices are not language-converted for every variant due to caching
Sitenotices are not language-converted for every variant due to caching
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.24rc
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
https://zh.wikivoyage.org/
: i18n
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-08 04:22 UTC by zhuyifei1999
Modified: 2014-09-02 09:10 UTC (History)
4 users (show)

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


Attachments

Description zhuyifei1999 2014-08-08 04:22:13 UTC
As reported at https://zh.wikivoyage.org/wiki/Wikivoyage:%E4%BA%92%E5%8A%A9%E5%AE%A2%E6%A0%88#.E4.B8.8A.E9.9D.A2.E7.9A.84.E5.91.8A.E7.A4.BA.E6.B2.92.E6.9C.89.E8.BD.89.E6.8F.9B.E6.AD.A3.E3.80.81.E7.B0.A1.E9.AB.94.E4.B8.AD.E6.96.87 , even if the user language and user language variant are changed, sitenotice still appears to be the same:
* https://zh.wikivoyage.org/zh-hans/Special:Blankpage?zh-hans
* https://zh.wikivoyage.org/zh-hant/Special:Blankpage?zh-hant

After some research, I've found out that in https://github.com/wikimedia/mediawiki-core/blob/6f022809880ed9ae330045b76723f9d14601916e/includes/Skin.php#L1552 , $needParse partly depends on whether [[MediaWiki:Sitenotice]] was actually changed. However, language converter works within the parser, which means that even if the wikitext was the same, after parsing, the text could actually change. It is recently that the variant of the cached version changed from zh to zh-tw, and some non-hant users have found that the sitenotice became unreadable for them.

As shown in https://github.com/wikimedia/mediawiki-core/blob/6f022809880ed9ae330045b76723f9d14601916e/includes/Skin.php#L1549 , a change to $wgRenderHashAppend (which defaults to '') could re-parse the page.
And due to above issues, I propose to append the variant name $wgRenderHashAppend while processing sitenotice.
Comment 2 Gerrit Notification Bot 2014-08-24 09:25:28 UTC
Change 156026 had a related patch set uploaded by Zhuyifei1999:
Add language code to sitenotice cache key

https://gerrit.wikimedia.org/r/156026
Comment 3 Gerrit Notification Bot 2014-09-02 09:10:31 UTC
Change 156026 abandoned by Zhuyifei1999:
Add language code to sitenotice cache key

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

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


Navigation
Links