| Summary: | Temperature of "zero" from weatheroffice.gc.ca results in blank in forecast | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Jason Benwell <jbenwell> |
| Component: | Weather widget | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | isma.af, kdedev, nate |
| Priority: | NOR | ||
| Version First Reported In: | 6.5.3 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/kdeplasma-addons/-/commit/36284870593d925e1ff073ab7fc99370e27f5509 | Version Fixed/Implemented In: | 6.5.5 |
| Sentry Crash Report: | |||
| Attachments: | Screenshot showing blank temperature forecast | ||
|
Description
Jason Benwell
2025-12-02 14:32:17 UTC
I can see in the code how this would happen. In this case we should have a "-" or "?" character or say "unknown" or something. (In reply to Nate Graham from comment #1) > I can see in the code how this would happen. In this case we should have a > "-" or "?" character or say "unknown" or something. I believe it should report 9C as the highnor low in this case - is that right? Thanks! (In reply to Jason Benwell from comment #2) > (In reply to Nate Graham from comment #1) > > I can see in the code how this would happen. In this case we should have a > > "-" or "?" character or say "unknown" or something. > > I believe it should report 9C as the highnor low in this case - is that > right? Thanks! Sorry for the typos. I believe it should report 0C as the high or low in this case - is that right? A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/964 Git commit ee9f587cbd8b00b581f44a959f421696a86ffe8b by Ismael Asensio. Committed on 23/12/2025 at 15:06. Pushed by iasensio into branch 'master'. applets/weather: Fix visibility for zero degrees temperatures When showing temperatures in the forecast delegates, we cannot check for "trueness" of the value in order to hide it, as 0ºC (or even 0ºF) are valid temperatures. Check instead if the value has been set. Javascript's `isNaN()` is very handy, since it works for both `nan`vand `undefined`. FIXED-IN: 6.5.90 M +4 -4 applets/weather/qml/ForecastView.qml https://invent.kde.org/plasma/kdeplasma-addons/-/commit/ee9f587cbd8b00b581f44a959f421696a86ffe8b Git commit 36284870593d925e1ff073ab7fc99370e27f5509 by Ismael Asensio. Committed on 23/12/2025 at 19:06. Pushed by iasensio into branch 'Plasma/6.5'. applets/weather: Fix visibility for zero degrees temperatures When showing temperatures in the forecast delegates, we cannot check for "trueness" of the value in order to hide it, as 0ºC (or even 0ºF) are valid temperatures. Check instead if the value has been set. Javascript's `isNaN()` is very handy, since it works for both `nan`vand `undefined`. FIXED-IN: 6.5.90 (cherry picked from commit ee9f587cbd8b00b581f44a959f421696a86ffe8b) Co-authored-by: Ismael Asensio <isma.af@gmail.com> M +4 -4 applets/weather/qml/ForecastView.qml https://invent.kde.org/plasma/kdeplasma-addons/-/commit/36284870593d925e1ff073ab7fc99370e27f5509 |