Calculator does not show thousands seperator despite being chosen in preferences

i can confirm this, choosing show thousand separator doesn't make any difference.

This is a little bit weird because for me works fine, please see attached screenshot.

I think this is a little more important than "Low." I have to start college algebra next semester, and a thousands separator would be nice...

This was working in Lucid, marking as regression.

Fixed upstream

Patch that fixes thousands separator functionality

Accepted gcalctool into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Once fixed, we are going to create a regression test using Mago for this issue.

Tested with gcalctool (5.32.0-0ubuntu2)
In English locale, now we got the thousands separator correctly. In Spanish, though, it does not put it. I guess it because the locale for Spanish does not include separators for the numbers, but it feels a bit weird to select an option that does not have any effects.

Also, I run the mago regression tests on it, and they all passed.

Thanks Ara, I've opened a bug for that here https://bugzilla.gnome.org/show_bug.cgi?id=632182

This patch exposed a locale bug (bug #660798) that is present in (at least) ru_RU.UTF-8. I've uploaded 5.32.0-0ubuntu3 with the attached patch that fixes the problem

Accepted gcalctool into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

i'll give it a try tomorrow at the office. thanks :)

doesn't work on the pt_PT locale

works great in english language. thanks dude :)

That works only for languages which have the property "thousands_sep" defined in LC_NUMERIC and the point that Ara and Sergio are raising is fair.

The Unix locales is not the _truth_ and the user should be able to set a grouping separator if he wants to even if none is defined in the Unix locale.

What does the standardization organization says:
"Therefore the space is recommended in the SI/ISO 31-0 standard,[8] and the International Bureau of Weights and Measures states that "for numbers with many digits the digits may be divided into groups of three by a thin space, in order to facilitate reading." (summary from wikipedia http://en.wikipedia.org/wiki/Decimal_mark#Digit_grouping feel free to read the full iso documentation if you have problem sleeping)

Moreover, the Unicode consortium (http://cldr.unicode.org/) defines in its Unicode Common Locale Data Repository a grouping separator for es_ES - '.' - and pt_PT - 'space'.

So, I think that a full fix for this, without using another repository than the Unix locale, would be to use the value for the grouping separator as defined in LC_NUMERIC and if none or empty then fallback to 'space' which is internationally recognized.

What do you think ?

Thanks Jean-Baptiste, sounds good to me, I've updated the upstream bug to do this.

Works as intended in Maverick. Marking Fix Released.

lsb_release -rd
Description: Ubuntu 10.10
Release: 10.10

 apt-cache policy gcalctool
gcalctool:
  Installed: 5.32.0-0ubuntu4
  Candidate: 5.32.0-0ubuntu4
  Version table:
 *** 5.32.0-0ubuntu4 0
        500 http://us.archive.ubuntu.com/ubuntu/ maverick-updates/main i386 Packages
        100 /var/lib/dpkg/status
     5.32.0-0ubuntu1 0
        500 http://us.archive.ubuntu.com/ubuntu/ maverick/main i386 Packages