launchpad misidentifies % as string formatting code

Bug #1243315 reported by Marcos Lans
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Invalid
Undecided
Unassigned
intltool
New
Undecided
Unassigned
checkbox (Ubuntu)
Won't Fix
Low
Unassigned
Revision history for this message
Marcos Lans (markooss) wrote :
Revision history for this message
Daniel Manrique (roadmr) wrote :

The checkbox.pot file has this:

#: ../scripts/gateway_ping_test:257
#, c-format
msgid "Connection established, but lost {}% of packets"
msgstr ""

This is wrong as the string is clearly python-format. I think that's what's causing the problem. I need to look at how intltool extracts strings from that file.

Thanks for reporting this!

Changed in checkbox (Ubuntu):
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Daniel Manrique (roadmr) wrote :

OK, the % is causing intltool-update to mistake the string for a c-format one. If it's removed, then it doesn't get a type specifier. Also, if the file is renamed gateway_ping_test.py, it gets correctly identified as python-format.

I'll look at which option is better for our purposes.

Revision history for this message
Marcos Lans (markooss) wrote :

What about this bug guys?

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I'll try to fix that though I don't know if we can really fix it without patching intltools.

summary: - Launchpad systema doesn't accept a correct translation
+ launchpad misidentifies % as string formatting code
Changed in checkbox (Ubuntu):
importance: High → Low
Revision history for this message
William Grant (wgrant) wrote :

The .po that was given to Launchpad said the string was c-format, so Launchpad is behaving correctly here.

Changed in launchpad:
status: New → Invalid
Revision history for this message
Marcos Lans (markooss) wrote :

I tested Checkbox in Ubuntu 14.04 (galician language) but it displays english version though is full translated into galician.

Revision history for this message
Jeff Lane  (bladernr) wrote :

Setting to Wont Fix as checkbox is no longer part of the desktop install and this is now out of scope.

Changed in checkbox (Ubuntu):
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.