Bug 506810 - Settings are lost
Summary: Settings are lost
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Weather widget (other bugs)
Version First Reported In: 6.4.80
Platform: Other Linux
: HI normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
: 510014 (view as bug list)
Depends on:
Blocks:
 
Reported: 2025-07-09 15:47 UTC by daron439
Modified: 2025-09-29 08:56 UTC (History)
7 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description daron439 2025-07-09 15:47:58 UTC
SUMMARY
After updating to https://invent.kde.org/plasma/kdeplasma-addons/-/commit/ea37d3e98da794f7b617b8761c259905fe263ce2, all weather applets now display "Configure..." instead of the previously configured location.

STEPS TO REPRODUCE
1. Update kdeplasma-addons to ea37d3e98da794f7b617b8761c259905fe263ce2

OBSERVED RESULT
All weather applets require reconfiguration; existing settings are not preserved.

EXPECTED RESULT
Applets should display the previously configured location.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: 
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 Marco Martin 2025-07-10 14:02:25 UTC
If you built from git without cleaning the build directory, there isa known problem for plasmoids updating from qml files on disk to compiled qml files: the old files are still taken and gets mixed with the new stuff, breaking the widget (this will not happen to who updates from packages).

Try to manually delete on the installed plasma prefix, the folder share/plasma/plasmoids/org.kde.weather
Comment 2 daron439 2025-07-10 14:38:46 UTC
I don't have that folder. On 344f25aef2803ee6b7419d131f9fceaed1a59783(kdeplasma-addons), this is a snippet from ~/.config/plasma-org.kde.plasma.desktop-appletsrc:
[Containments][2][Applets][30][Configuration][WeatherStation]
source=bbcukmet|weather|City of London, Greater London, GB|2643741

After updating to 5b3082e914c493a5e6f237c9fe7e4641647e16b9(kdeplasma-addons) and reconfiguring the applet:
[Containments][2][Applets][30][Configuration][WeatherStation]
placeDisplayName=London, Greater London, GB
placeInfo=London, Greater London, GB|2643743
provider=bbcukmet
providerCredit=BBC Weather
source=bbcukmet|weather|City of London, Greater London, GB|2643741
Comment 3 Nate Graham 2025-07-10 15:39:39 UTC
I can confirm this myself. Upon building that commit and restarting plasmashell, my configured weather widget became unconfigured.
Comment 4 Bogdan Onofriuchuk 2025-07-10 16:04:29 UTC
The problem here is due to the porting from the dataengine. The overall structure of the weather applet was refactored with some changes to how the applet requests locations and forecasts. To handle this you just need to reconfigure the applet and it will be working as before.
Comment 5 Nate Graham 2025-07-10 16:07:36 UTC
Obviously. :) But losing user settings isn't acceptable; we have to fix this.
Comment 6 Bogdan Onofriuchuk 2025-07-10 21:01:37 UTC
Created MR: https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/882 
For some reason it is not shown here.
Comment 7 Bug Janitor Service 2025-07-28 10:43:42 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/892
Comment 8 Nate Graham 2025-08-22 16:55:33 UTC
Where are we with this issue? I see two merge requests, one huge, and one controversial. Neither seems ideal. Can we instead revert the breaking change?
Comment 9 Bogdan Onofriuchuk 2025-08-22 17:50:43 UTC
> Where are we with this issue?
I have created the second MR which handle old format and convert it to a new. Waiting for review

> I see two merge requests, one huge, and one controversial.
The second is just handle the old format and return new instead which is than saved instead of the old. 

> Can we instead revert the breaking change?
That was done in the porting from the dataengine MR. So to revert it we need to revert that MR.
Comment 10 Bogdan Onofriuchuk 2025-08-26 21:05:51 UTC
Git commit f9ae6e1e44880c492c35baf139a7cc4f245c96ee by Bohdan Onofriichuk.
Committed on 26/08/2025 at 20:31.
Pushed by littlesweet into branch 'master'.

applets/weather: respect previous config source

M  +11   -0    applets/weather/ions/bbcukmet/ion_bbcukmet.cpp
M  +3    -0    applets/weather/ions/bbcukmet/ion_bbcukmet.h
M  +11   -0    applets/weather/ions/dwd/ion_dwd.cpp
M  +3    -0    applets/weather/ions/dwd/ion_dwd.h
M  +49   -4    applets/weather/ions/envcan/ion_envcan.cpp
M  +4    -0    applets/weather/ions/envcan/ion_envcan.h
M  +14   -0    applets/weather/ions/noaa/ion_noaa.cpp
M  +4    -0    applets/weather/ions/noaa/ion_noaa.h
M  +12   -0    applets/weather/ions/wetter.com/ion_wettercom.cpp
M  +4    -0    applets/weather/ions/wetter.com/ion_wettercom.h
M  +3    -0    applets/weather/main.xml
M  +19   -1    applets/weather/qml/main.qml
M  +14   -0    applets/weather/weatherdata/station.cpp
M  +5    -0    applets/weather/weatherdata/station.h

https://invent.kde.org/plasma/kdeplasma-addons/-/commit/f9ae6e1e44880c492c35baf139a7cc4f245c96ee
Comment 11 Akseli Lahtinen 2025-09-29 08:56:59 UTC
*** Bug 510014 has been marked as a duplicate of this bug. ***