Bug 224745

Summary: De-installing of Plasma-Themes does not actually remove them (GHNS)
Product: [Unmaintained] plasma4 Reporter: Marc Schiffbauer <mschiff>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: annma, asraniel, davide.bettio, gladhorn, toddrme2178
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Marc Schiffbauer 2010-01-29 12:28:25 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

To reproduce:

* Install a plasma theme via GHNS
* Now ist in the list of plasma themes and can be used
* De-Install it via GHNS
-> Its still in the list even after a restart of system-settings
Comment 1 Marc Schiffbauer 2010-02-02 02:43:27 UTC
Still broken in 4.4RC3
Comment 2 Anne-Marie Mahfouf 2010-02-02 12:53:45 UTC
Why is this filed to widget-explorer?
Does this happen also for all GHNS areas such as wallpapers or comic for example? If so it's to be filed for GHNS (kdelibs -> knewstuff3)
Comment 3 Marc Schiffbauer 2010-02-02 15:44:24 UTC
Can you please set the correct component then?

I now tested wallpapers, which seem to work as expected (Install / Uninstall)
Comment 4 Anne-Marie Mahfouf 2010-02-02 18:44:52 UTC
It's not kdelibs/knewstuff if some implementation works.
The bug should be assigned to the Workspace Theme component which is not a bugzilla component. 
I think the theme cannot be removed as it appears in the list.
I talked to the devel and he said he'll look at it.
Reassigning to plasma general et CCing the devel
Comment 5 Frederik Gladhorn 2010-02-02 20:19:38 UTC
plasma-themes.knsrc sets custom install/uninstall commands:
InstallationCommand=plasmapkg -t theme -i %f
UninstallCommand=plasmapkg -t theme -r %f

in %f it gets passed the url of the downloaded file
the plasmapkg uninstall command expects the package name not url for removal

looking at plasmapkg --help
  -i, --install <path>      Install the package at <path>
  -r, --remove <name>       Remove the package named <name>

so this can never have worked
Comment 6 Beat Wolf 2010-05-11 12:49:46 UTC
So what needs to be done to fix this? is it something simple we can just CC the person who coded it?
Comment 7 Marc Schiffbauer 2010-06-26 11:42:34 UTC
Still broken in 4.4.90
Comment 8 Todd 2011-01-18 22:29:41 UTC
Still broken in 4.6 RC 1.
Comment 9 Todd 2011-07-31 07:51:04 UTC
This is fixed now.