false positive: E201: invalid variables: %28e

[willkg]

dennis version 0.7
>>> Working on: /Users/rob/git/kuma/locale/it/LC_MESSAGES/django.po
E201: invalid variables: %28e
1029:#: kuma/demos/__init__.py:237
1030:msgid "http://en.wikipedia.org/wiki/Canvas_element"
1031:msgstr "http://it.wikipedia.org/wiki/Canvas_%28elemento_HTML%29"

That's wrong. For python-format, if there are no vars in the msgid, then we don't care about the msgstr.

[robhudson]

I see you fixed it, but what about something like which I could see easily happening as an extension of this case:

msgid = "Go to this %(adjective)s URL: http://en.wikipedia.org/wiki/Canvas_element"
msgstr = "Go to this %(adjective)s URL: http://it.wikipedia.org/wiki/Canvas_%28elemento_HTML%29"

[willkg]

That'll raise an error:

Python 2.7.10 (default, Sep 24 2015, 17:50:09) 
[GCC 5.1.1 20150618 (Red Hat 5.1.1-4)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> foo = "Go to this %(adjective)s URL: http://it.wikipedia.org/wiki/Canvas_%%28elemento_HTML%%29" % {'adjective': 'foo'}
>>> foo = "Go to this %(adjective)s URL: http://it.wikipedia.org/wiki/Canvas_%28elemento_HTML%29" % {'adjective': 'foo'}
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: not enough arguments for format string
>>> 

They need to double the percent:

msgstr: "Go to this %(adjective)s URL: http://it.wikipedia.org/wiki/Canvas_%%28elemento_HTML%%29"

[robhudson]

👍