Bug 462836 - Krunner time plugin shows only one result out of different identically named time zones
Summary: Krunner time plugin shows only one result out of different identically named ...
Status: RESOLVED FIXED
Alias: None
Product: krunner
Classification: Plasma
Component: general (other bugs)
Version First Reported In: 5.26.4
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-12-10 04:13 UTC by radimir.cacic09
Modified: 2023-02-14 17:09 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.27
Sentry Crash Report:


Attachments
Demonstration (244.47 KB, image/png)
2022-12-10 04:13 UTC, radimir.cacic09
Details

Note You need to log in before you can comment on or make changes to this bug.
Description radimir.cacic09 2022-12-10 04:13:18 UTC
Created attachment 154469 [details]
Demonstration

SUMMARY

Krunner doesn't display correctly the current time in PST. (my system is in CET)

STEPS TO REPRODUCE
1. Switch your timezone to CET
2. Open krunner
3. Type time PST
4. Observe my local time 10/12/2022 5:05 AM
5. Observe Krunner showing that it's 10/12/2022 12:05 AM in PST

However this isn't correct.  CET is 9 hours currently ahead of PST.
This means that it's 09/12/2022 8:05 in PST.

OBSERVED RESULT

Krunner shows that it's 10/12/2022 12:05 AM in PST.

EXPECTED RESULT

I'd expect time and date to be 09/12/2022 20:05 (8 PM) PST.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.26
KDE Plasma Version: 5.26.4
KDE Frameworks Version: 5.100.0
Qt Version: 5.15.7
Comment 1 Natalie Clarius 2022-12-10 18:07:17 UTC
Looks like there's two timezones abbreviated "PST" according to the Qt time zone library, one in North America with the time you're looking for and one associated with Asia/Manila which is what KRunner shows you.  

I think we should show both results in such cases, ideally disambiguated.
Comment 2 Bug Janitor Service 2022-12-11 08:23:06 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/292
Comment 3 radimir.cacic09 2022-12-13 05:18:25 UTC
Dang! That was lightning fast.
I really appreciate it Natalie!
Comment 4 radimir.cacic09 2023-02-14 16:59:40 UTC
Fixed in Plasma 5.27. Thanks again, Natalie!