Jump to content

PLURAL-Support added

PLURAL-Support added

Hi,

I've added a support for {{PLURAL...}} in the Toolserver-I18N but the code is not in the live version yet because I want that another developer review the code (so please use PLURAL not yet).

The code with the changes is under https://svn.toolserver.org/svnroot/jan/trunk/TS_I18N/. A complete diff-file is in the directory but the main changes are in TsIntuition.php and in the new file language/MessagesFunctions.php.

Jan Luca17:15, 27 June 2011

Thank you so much for working on this.

I'll check it out this weekend. Looks good from first look!

If I forget, feel free to bug me about it.

Krinkle18:27, 28 June 2011

Hi Krinkle,

I've added a error support for the messages functions so please see at the both patches in the SVN-directory.

Jan Luca19:50, 29 June 2011
 

I have added the PLURAL support to Wikimedia SVN.

For translator: The PLURAL support is similar to the MediaWiki one. It support the language-specified forms with more / less than two froms. To use PLURAL you call (like in MediaWiki)

For developers: To use PLURAL in your messages, please use the "parsemag" option of msg() (for support use the value TRUE, default value is FALSE).

Jan Luca08:38, 5 July 2011
 

Thanks. I did look at it but not a full review. Committing it was the right call, I would've responded here if there was a problem in the patch you linked to.

I'll push this to the sandbox tonight and deploy to the live /intuition/ shortly after.

Krinkle17:45, 6 July 2011

Plural is not working anymore. Have a look at this page, for example.

Rancher20:14, 3 October 2011

I see no problem. [1] seems to work.

Jan Luca (talk)19:21, 30 October 2011

In Serbian, it doesn't. Take a look at this, for example.

Rancher (talk)12:56, 31 October 2011
 
 

Whether it works in a certain tool is not a valid test, use the Demos instead.

Tools may be outputting a raw message. That was the case for the Intuition dashboard. That was fixed in the mean time:

Aside from that, when using the demo:

There is a warning about ../LanguageConverter.php missing. That is a bug, reported here:

Thanks!

Krinkle (talk)23:09, 4 March 2012