unable to remove an account under certain locales

[Steve Dodier-Lazaro]

The "delete selected account" button doesn't seem to work at all.

See the log from console :

steve@steve-laptop:~$ wxbanker

Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/wxbanker/accountlistctrl.py", line 443, in onRemoveButton
    dlg = wx.MessageDialog(self, warningMsg%account.Name, _("Warning"), style=wx.YES_NO|wx.ICON_EXCLAMATION)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 9: ordinal not in range(128)

[Steve Dodier-Lazaro (sidi)]

I tracked the issue down a little and i found where the bug was coming from :

Python fails to parse the string that should be written in the popup, because it's encoding isn't set, which makes Python use the ASCII encoding. And unfortunately, languages such as french have characters outside of ASCII.

The application should be set to UTF-8 in order to fix this kind of problems.

[Michael Rooney (mrooney)]

Thanks for the report. I guess I need to be more defensive and figure
out where to explicitly encode to perhaps utf-8. One challenge is that
utf-8 might not always be the right encoding; I am not quite sure how
to detect this.

[Steve Dodier-Lazaro (sidi)]

UTF-8 supports almost every alphabet of the world, and i think it's the one used by default everywhere. I don't know either about how to detect it, but the default definately shouldn't be ANSI, which is only suitable for english characters :)

[Steve Dodier-Lazaro (sidi)]

This patch seems to solve the issue here, related to the way the attention strings are translated. I also seem to be able to create and delete accounts with UTF-8 names.

[Michael Rooney (mrooney)]

If all you do is change the single quotes to double quotes it works, but this is only because there is no longer a translation for that string :) So, I don't think that patch would actually address the issue; I suspect if there was a translation for that new string it would do the same thing.

[Michael Rooney (mrooney)]

Okay fixed as of r224. After asking in #python I just need to add a .decode("utf-8") to the translated warningMsg before formatting with the account.Name. Thanks again for this catch!

[Steve Dodier-Lazaro (sidi)]

Well, it's been a long walk :)

I actually tried unicode(AccountName, UTF8) but it wasn't working here, saying it couldn't be parsed to UTF8.

The fix works here too !

[Karel Kolman (kolmis)]

Michael, wouldn't it be better to tell gettext to always return unicode strings ?

i.e. replace

gettext.install(APP, DIR)
with
gettext.install(APP, DIR, unicode=True)

in localization.py, that way _() calls gettext.ugettext() method.

[Michael Rooney (mrooney)]

Thanks for the suggestion Karel, let me investigate that. That might be a better solution indeed!

[Michael Rooney (mrooney)]

Thanks Karel, that works indeed, I will use that approach instead!

[Steve Dodier-Lazaro (sidi)]

Now having trouble with the "Send to calculator" due to the new french translation.

[Michael Rooney (mrooney)]

Since this bug was originally about just removing an account and was targeted for 0.5 and fixed, I'll mark it as such. Steve, does the calculator bug still exist? If so let's make a new bug and target for 0.6.

[Michael Rooney (mrooney)]

I confirmed it and created https://bugs.launchpad.net/wxbanker/+bug/375308. Karel, maybe you know the way to fix it. In the description I mention that wrapping the return of float2str calls in unicode seems good, and fixes the issue, but fails a test with the traceback given. Hmm!