Bug 423096

Summary:	country specific currency conversion
Product:	[Plasma] krunner	Reporter:	johnathan <testing1237a-c>
Component:	converter	Assignee:	Alexander Lohnau <alexander.lohnau>
Status:	RESOLVED FIXED
Severity:	normal	CC:	alexander.lohnau, plasma-bugs-null
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Other
OS:	Other
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/commit/edc64d04a1e569d7032c41e6ee0ebf59833c26f2	Version Fixed/Implemented In:	5.19.3
Sentry Crash Report:

Description johnathan 2020-06-17 11:21:32 UTC

there is a setting, "Regional setting" under format. 
I have set mine to "india" and it works as advertised fine.

A quality of life improvement i am suggesting is in krunner currency conversion. Currently if i type "$100" into krunner, it shows calculator and "EUR 81.76614881439084". 
I am suggesting, the currency conversion should pick country from regional settings and in my case, give me conversion in INR rupees instead of euro because that is what i am most probably going to use.

Comment 1 Alexander Lohnau 2020-06-18 05:59:30 UTC

Thanks for reporting this is actually a feature, but considering that it isn't working for you is a bug :-)

I will get into writing a fix.

Comment 2 Alexander Lohnau 2020-06-18 06:42:34 UTC

If you have selected a region under formats, but not detailed settings checked it does not work properly. Otherwise it does.

Comment 3 Alexander Lohnau 2020-06-18 07:10:56 UTC

Could you please run:
>printenv |egrep "^(LC|LANG)"
and post the output?

I changed my locale from german to india and after logout this was my result:

LC_MEASUREMENT=de_DE.UTF-8
LC_PAPER=de_DE.UTF-8
LC_MONETARY=de_DE.UTF-8
LANG=en_IN.UTF-8
LC_NAME=de_DE.UTF-8
LC_ADDRESS=de_DE.UTF-8
LC_NUMERIC=de_DE.UTF-8
LANGUAGE=
LC_TELEPHONE=de_DE.UTF-8
LC_IDENTIFICATION=de_DE.UTF-8
LC_TIME=de_DE.UTF-8

Comment 4 Alexander Lohnau 2020-06-25 13:41:40 UTC

Git commit edc64d04a1e569d7032c41e6ee0ebf59833c26f2 by Alexander Lohnau.
Committed on 25/06/2020 at 13:18.
Pushed by alex into branch 'Plasma/5.19'.

Fix broken ENV variables for detailed settings
Related: bug 417070, bug 327757

M  +21   -10   startkde/startplasma.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/edc64d04a1e569d7032c41e6ee0ebf59833c26f2

Comment 5 johnathan 2020-07-07 12:52:56 UTC

$ printenv |egrep "^(LC|LANG)"
LANGUAGE=en_GB
LANG=en_IN.UTF-8