Bug 236258 - Plasma/Superkaramba doesn't work in 4.4.3
Summary: Plasma/Superkaramba doesn't work in 4.4.3
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-04 02:07 UTC by WiseLord
Modified: 2010-05-04 05:19 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
This patch resolves a problem. (665 bytes, text/plain)
2010-05-04 02:10 UTC, WiseLord
Details

Note You need to log in before you can comment on or make changes to this bug.
Description WiseLord 2010-05-04 02:07:32 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

Today I installed kde-4.4.3 and found that my favourite superkaramba vidget doesn't work as plasmoid. Plasma couldn't find path of plasmoid (but it was in plasmoid list).

Error was in kdelibs package, in file kdelibs-4.4.3/plasma/applet.cpp.
Here is the differences between 4.4.2 and 4.4.3 in this file:

--- kdelibs-4.4.2/plasma/applet.cpp	2010-03-26 19:24:32.000000000 +0200
+++ kdelibs-4.4.3/plasma/applet.cpp	2010-04-29 22:52:24.000000000 +0300
@@ -2585,7 +2585,8 @@
         // find where the Package is
         QString path = packagePath;
         if (path.isEmpty()) {
-            path = KStandardDirs::locate("data", "plasma/plasmoids/" + appletDescription.pluginName() + '/');
+            path = KStandardDirs::locate("data", "plasma/plasmoids/" + appletDescription.pluginName() + "/metadata.desktop");
+            path.replace(QString("/metadata.desktop"),QString("/"));
         } else if (!path.endsWith('/')) {
             path.append('/');
         }

When I recompiled kdelibs-4.4.3 with this file from 4.4.2 all became work fine.
Comment 1 WiseLord 2010-05-04 02:10:54 UTC
Created attachment 43206 [details]
This patch resolves a problem.

Patch returnes applet.cpp from kdelibs-4.4.3 to its 4.4.2 variant.
Comment 2 Aaron J. Seigo 2010-05-04 05:19:10 UTC
SVN commit 1122503 by aseigo:

Riddell's commit to fix broken installations (and i think it's still something of a bad hack, tbh) broke ALL OTHER WIDGET TYPES. fail.
this fixes that.
BUG:236258


 M  +7 -3      applet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1122503