819534 - l10n should (?) support plurals and placeholders in the same `get`

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, enhancement, P2)

Description

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

Should this work?  

(oh.my plural is:  'you are afraid of %s -- oh my!')

require("l10n").get("oh.my", 3, 'lions, and tigers, and bears')

I see that it branches in the code between 'plural' and not, but I have no idea of what the expected behaviour of old StringBundles were.

Comment 1

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Alex, what do you think?

Comment 2

[Gregg Lind (Fx Strategy and Insights - Shield - Heartbeat )]

(Not to bugsit, but pluralization in general is complicated and quite messy!  

Maybe a second function  `getPlural(id,count,...)` makes it less gross?  Having to specify '1' all the time, or being worried that if the 2nd arg is an int, that it will trigger plural handling seems gross too!

Suggestion:  

get -> always 1.
getPlural -> requires count arg

)

Comment 3

[Alexandre Poirot [:ochameau]]

That's a reasonable request. I decided for the first version of the localization module for jetpack to 'KISS' and so ignored complex usages of plurals.
But it would be worth to continue the development of localization support for the SDK.
I'm currently considering syncing with l10n effort being done in Gaia (UI for Firefox OS) that embrace HTML localization.
For the story, me and kaze [the webl10n developer] started working together to come up with similar approach. He never stopped tuning his library, whereas I jumped on other tasks...
So it will be matter of pondering the compability cost in order to upgrade with last webl10n version.

Comment 4

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562