Last modified: 2014-08-04 10:53:18 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 T50956, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48956 - Can't override optional message in all languages with local customisation
Can't override optional message in all languages with local customisation
Status: NEW
Product: MediaWiki
Classification: Unclassified
Internationalization (Other open bugs)
unspecified
All All
: Normal normal with 2 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 1495
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-29 23:18 UTC by MF-Warburg
Modified: 2014-08-04 10:53 UTC (History)
16 users (show)

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


Attachments

Description MF-Warburg 2013-05-29 23:18:19 UTC
Compare https://incubator.wikimedia.org/wiki/Special:Contributions/QuincyMcc?uselang=en and https://incubator.wikimedia.org/wiki/Special:Contributions/QuincyMcc?uselang=de
and notice the missing content of MediaWiki:Sp-contributions-footer.

On other wikis (meta, en) this problem does not seem to exist.
Comment 1 Nemo 2013-05-29 23:40:49 UTC
This is the normal behaviour for locally overridden messages.
I guess this needs the followup to https://gerrit.wikimedia.org/r/#/c/57518/ with the completion of the plan as per bug 46579 comment 13 (1), i.e. ability to override a message in all languages by writing on /en?
Comment 2 MF-Warburg 2013-05-29 23:46:17 UTC
I know that this is the normal behaviour for "normal" messages (and complained about that in bug 1495).
But I wonder why this [apparently optional message, since it's empty by default and customized by wikis on their own] (so far) worked well? (i.e. it got shown even when not using the English interface, as it's still(?) the case on Meta and elsewhere).
Comment 3 Niklas Laxström 2013-05-30 06:54:43 UTC
Maybe different versions of software deployed.
Comment 4 Herby 2013-05-30 07:36:15 UTC
This has now hit Meta and makes dealing with cross wiki issues - disruption/spamming and the like FAR harder.  Getting it sorted soon would be appreciated.
Comment 5 Nemo 2013-05-30 07:45:40 UTC
(In reply to comment #2)
> But I wonder why this [apparently optional message, since it's empty by
> default
> and customized by wikis on their own] (so far) worked well?

Perhaps the key is "empty by default", i.e. defined as "-" beside being ignored (rather than optional)? How to deal with blank or "-" messages is a tricky thing.
Comment 6 Tyler Romeo 2013-05-30 15:20:40 UTC
*sigh* don't tell me we're going to have to revert the fallback change again.

(In reply to comment #5)
> (In reply to comment #2)
> > But I wonder why this [apparently optional message, since it's empty by
> > default
> > and customized by wikis on their own] (so far) worked well?
> 
> Perhaps the key is "empty by default", i.e. defined as "-" beside being
> ignored
> (rather than optional)? How to deal with blank or "-" messages is a tricky
> thing.

Yes, this is the issue. I tested this and running "Language::factory('de')->getMessage('sp-contributions-footer')" returns the message '-', which MessageCache would then treat as a found message.

But what I'm having trouble understanding is why that is happening. The 'sp-contributions-footer' message isn't defined anywhere in MessagesDe.php, so where is the '-' coming from? The message should come back as non-existent, allowing it to fall back onto the content language.
Comment 7 Niklas Laxström 2013-05-30 16:34:47 UTC
I guess it comes from MessagesEn.php because of the fallback merging on l10n cache.


On the other hand, it imagine it wouldn't be too hard to add the logic I proposed earlier in code review:

If MediaWiki:A exists, skip l10n cache.

It's not a perfect solution and one step back wrt to the goals of this patchet, but in any case there will a big migration later to separate overrides from customisations.

Any other ideas?
Comment 8 Andre Klapper 2013-07-08 11:04:46 UTC
Tyler: Any other ideas? (refering to comment 7)
Comment 9 Tyler Romeo 2013-07-08 11:09:44 UTC
I'm not too sure. I mean, in the original links from comment 1, the issue doesn't appear to occur anymore. Also, my patch didn't change the current message fallback process; it merely appended to it.

As for how to fix the bigger issue of actually handling language fallbacks properly, unfortunately that can't be done without somehow being able to interpret the results of the CDB cache.
Comment 10 MF-Warburg 2013-07-08 13:04:40 UTC
(In reply to comment #9)
> I'm not too sure. I mean, in the original links from comment 1, the issue
> doesn't appear to occur anymore.

This is only because I created https://incubator.wikimedia.org/wiki/MediaWiki:Sp-contributions-footer/de as a workaround to prevent the complete unusability of the interface for me.
On e.g. https://incubator.wikimedia.org/wiki/Special:Contributions/QuincyMcc?uselang=fr it still appears.
Comment 11 Gerrit Notification Bot 2013-07-09 23:18:54 UTC
Change 72867 had a related patch set uploaded by Parent5446:
Complete usage of message fallback chain

https://gerrit.wikimedia.org/r/72867
Comment 12 Andre Klapper 2014-08-04 10:53:18 UTC
For the records, patch was -1'ed and needs improvement.

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


Navigation
Links