640179 - [webby] Update verbatim extraction

Status: RESOLVED FIXED

(Webtools Graveyard :: Elmo, defect)

Description

[Zibi Braniecki [:zbraniecki][:gandalf]]

Currently extracting locales from verbatim does not work for ab-CD locales and it also catches en-US.

Comment 1

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: patch

Comment 2

[Zibi Braniecki [:zbraniecki][:gandalf]]

landed http://hg.mozilla.org/l10n/django-site/rev/7da278694a02

Comment 3

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: patch 2

split sections in the config to verbatim and svn cause we need separate mappings for svn.

Comment 4

[Zibi Braniecki [:zbraniecki][:gandalf]]

Attachment: patch 2 v2

updated patch. Need only one mapping system.

Comment 5

[Staś Małolepszy :stas]

Comment on attachment 518079 [details] [diff] [review]
patch 2 v2

>+    if len(locale_mapping)==0:
>+        _get_locale_mapping()

>+    if len(locale_mapping)==0:
>+        _get_locale_mapping()

You can just do

    if locale_mapping:
        _get_locale_mapping()

And I'd probably change the name of the cfg files from verbatim_mapping.cfg to locale_codes.cfg.

r=me otherwise.

Comment 6

[Zibi Braniecki [:zbraniecki][:gandalf]]

http://hg.mozilla.org/l10n/django-site/rev/6cd453c8657a
http://hg.mozilla.org/l10n/django-site/rev/d274ac441610