Last modified: 2011-06-06 21:25:54 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 T31274, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 29274 - Message class ignores $wgForceUIMsgAsContentMsg
Message class ignores $wgForceUIMsgAsContentMsg
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Internationalization (Other open bugs)
1.17.x
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 26676
  Show dependency treegraph
 
Reported: 2011-06-04 23:03 UTC by Platonides
Modified: 2011-06-06 21:25 UTC (History)
2 users (show)

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


Attachments

Description Platonides 2011-06-04 23:03:38 UTC
wfMessage('mainpage')->inContentLanguage() always gives the content language, even if it is overrriden by $wgForceUIMsgAsContentMsg

php maintenance/eval.php
> $wgContLang = Language::factory('en');
> $wgLang = Language::factory('fr');
> $wgForceUIMsgAsContentMsg[] = 'mainpage';
> echo wfMsgForContent('mainpage');
Accueil

> echo wfMessage('mainpage')->inContentLanguage()->text();
Main Page
Comment 1 Niklas Laxström 2011-06-06 07:00:37 UTC
I can fix this.
Comment 2 Krinkle 2011-06-06 16:53:33 UTC
Is this a regression from 1.16 ?

(In reply to comment #1)
> I can fix this.

Awesome.
Comment 3 Niklas Laxström 2011-06-06 17:26:21 UTC
(In reply to comment #2)
> Is this a regression from 1.16 ?

Depends on if those kind of calls have been converted a lot.
Comment 4 Niklas Laxström 2011-06-06 17:55:41 UTC
r89585.
Comment 5 Platonides 2011-06-06 21:25:54 UTC
Great, thanks Niklas.

Krinkle, it was a regression for places that used WfMsgForContent and were changed to use wfMessage()->inContentLanguage().

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


Navigation
Links