Bug 159894

Summary: When plasma crashes on removing a plasmoid, remove it
Product: [Plasma] plasma4 Reporter: Iuri Fiedoruk <protomank>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED INTENTIONAL    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:

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.