I18n: the downloaded CVS summary is in mixed languages

[Ying Chun Guo]

Hi,

When I set the language in Chinese, and downloaded the CVS summary from the "Overview" page, I found the CVS file were in two langauges. The start date and end date of a period were described in Chinese. "Project ID" was translated in Chinese. Others were still in English.

----Here is an example----
Usage Report For Period:,九月. 17 2013,九月. 17 2013
项目ID:,d2c23998b9364d7d8c5242536f117e1f
Total Active VCPUs:,0
CPU-HRs Used:,0.00
Total Active Ram (MB):,0
Total Disk Size:,0
Total Disk Usage:,0.00
Instance Name,VCPUs,Ram (MB),Disk (GB),Usage (Hours),Uptime(Seconds),State

Regards
Daisy

[Victor Morales (electrocucaracha)]

After reviewing the code, I consider that this bug is related with a missing entries in the translation files

https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/overview/views.py#L34
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/admin/overview/templates/overview/usage.csv
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/overview/views.py#L33
https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/overview/templates/overview/usage.csv

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/51597

[Akihiro Motoki (amotoki)]

From the discussion in the review, the cause is the default target files of Django manage.py makemessages are *.html and *.py, and CSV files are not processed. The proposed patch is now updating run_tests.sh to add *.csv to the target files.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/51597
Committed: http://github.com/openstack/horizon/commit/0b03a7fdea3d6586ec2c22783e309bb75d68fdc5
Submitter: Jenkins
Branch: master

commit 0b03a7fdea3d6586ec2c22783e309bb75d68fdc5
Author: Lei Zhang <email address hidden>
Date: Mon Oct 14 22:00:34 2013 +0800

    Add extra extension file to makemessage command line

    specify the file extensions of the makemessage to *.html, *.txt, *.csv

    Closes-Bug: #1226910
    Change-Id: I115cd448249ac0b1b27a6b28fca3f16d6b2670a6

[Julie Pichon (jpichon)]

Should this updated command be backported (+ extra fix in commit 79ae2139bb) to Havana in order for us to generate more complete .po files before the next stable update, or do we want to avoid adding new strings to the Havana po files?

It seems to me - backporting to Havana wouldn't hurt since the strings are not translated now anyway. Translators who have the time and will to translate these new strings would then be free to do it if they so wish.

[Ying Chun Guo (daisy-ycguo)]

I prefer to backport.
There won't be many strings.

[Akihiro Motoki (amotoki)]

Agree to havana backport.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/59916

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/59916
Committed: http://github.com/openstack/horizon/commit/06e78a8bd7c7d3d777c152230f9b8c88d2dd0487
Submitter: Jenkins
Branch: stable/havana

commit 06e78a8bd7c7d3d777c152230f9b8c88d2dd0487
Author: Akihiro Motoki <email address hidden>
Date: Tue Dec 3 23:56:48 2013 +0900

    Add extra extension file to makemessage command line

    specify the file extensions of the makemessage to *.html, *.txt, *.csv

    Closes-Bug: #1226910
    (cherry picked from commit 0b03a7fdea3d6586ec2c22783e309bb75d68fdc5)

    The above fix introduces an extra bug when updating POT files, so
    the commit below is also cherry-picked.
    ----
    Fix warning in run_tests.sh --makemessages

    Specify --extension=html,txt,csv only for openstack_dashboard.
    For djangojs, --extension should be 'js'.
    This is the cause of the warning.

    Closes-Bug: #1243526
    (cherry picked from commit 79ae2139bb2f458de698aa116270428f54e22dc6)
    ----

    This patch does not update POT files for stable/havana.
    They will be updated in a single patch.

    Change-Id: I115cd448249ac0b1b27a6b28fca3f16d6b2670a6