Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Score is corrupted on systems with locale that uses comma as "floating point" delimeter #734

Open
v-fox opened this issue Apr 3, 2024 · 2 comments
Labels

Comments

@v-fox
Copy link

v-fox commented Apr 3, 2024

Some languages, like Russian, use comma instead of point, so trackma automatically substitutes points with commas in score field, only to complain with invalid score error. You can force it to take the score by manually putting point instead but then it immediately reverts it.

The correct way would be to take both and transparently convert to what remote server expects.
A workaround would be to always force point on all locales.

@FichteFoll
Copy link
Collaborator

FichteFoll commented May 1, 2024

Which UI are you using (and what's your exact locale output)?

@FichteFoll FichteFoll added the bug label May 1, 2024
@v-fox
Copy link
Author

v-fox commented May 1, 2024

You can force it to take the score by manually putting point instead but then it immediately reverts it.

I misremembered how workaround works. What you actually have to do is to force it to give Score is already at <X> message after which it does not check the input and lets you put whatever you want.

Which UI are you using (and what's your exact locale output)?

I used Qt and now migrated to GTK one due to system transition to Qt6, to drop fat PyQt5 dependency. Both are affected.

% locale
LANG=en_DK.UTF-8
LC_CTYPE=ru_RU.UTF-8
LC_NUMERIC=ru_RU.UTF-8
LC_TIME=ru_RU.UTF-8
LC_COLLATE=ru_RU.UTF-8
LC_MONETARY=ru_RU.UTF-8
LC_MESSAGES=en_GB.UTF-8
LC_PAPER=ru_RU.UTF-8
LC_NAME=ru_RU.UTF-8
LC_ADDRESS=ru_RU.UTF-8
LC_TELEPHONE=ru_RU.UTF-8
LC_MEASUREMENT=ru_RU.UTF-8
LC_IDENTIFICATION=ru_RU.UTF-8
LC_ALL=

Strangely, KDE/Plasma still forces Russian in apps by default, despite LANG and LC_MESSAGES.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants