Bug 159894 - When plasma crashes on removing a plasmoid, remove it
Summary: When plasma crashes on removing a plasmoid, remove it
Status: RESOLVED INTENTIONAL
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-26 14:57 UTC by Iuri Fiedoruk
Modified: 2008-07-05 19:15 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Iuri Fiedoruk 2008-03-26 14:57:20 UTC
Version:            (using KDE 4.0.2)
Installed from:    Ubuntu Packages

Steps to reproduce:
- get a bad plasmoid that crashes when removed from panel (I've had this trouble with network graph and changing resolution)
- try to remove it from the panel
- plasma crashes
- plasma restarts
- there is your plasmoid again! *loop*
it will never be removed this way. When removing a plasmoid, it should not return if plasma crashes, it should be removed from plasmarc *before* fromving it graphicaly from the panel, so if plasma crashes, it won't be there on restart :)

- but please don't do the same when adding a plasmoid, heheheheh ;)
Comment 1 Alex Merry 2008-07-05 12:18:30 UTC
This would be useful, but the implementation would need to be a bit hackish, given how saving currently works.  Probably should be re-visited for 4.1.x or 4.2.
Comment 2 Aaron J. Seigo 2008-07-05 19:15:04 UTC
this could easily lead to all sorts of interesting problems. if it the config is saved without that item in it, all the other items geometries will be wrong in the config file (there's be a "hole" in there). i can only imagine other similar odd issues would creep up.

it would also require that all applets, upon quitting do not touch their configuration object at all, otherwise it would just get recreated anyways. result: same problem, despite the hacks. we might be able to give to-be-deleted applets a dummy config object, but that isn't exactly foolproof either.

the real solution here is not crashing on deleting an applets.

in the case where this is what is happening, please report bugs against that item or remove it from the config file manually. perhaps at some point we'll provide a config file editor? but the suggested approach in this br is a too risky for my tastes.