Summary: | "set as background" export tool | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | John Calixto <john> |
Component: | Plugin-Generic-WallPaper | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | aspotashev, caulier.gilles, kde |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 6.2.0 |
Description
John Calixto
2008-05-15 07:51:55 UTC
With strace, I found the following errors when trying to set the background: [pid 8172] write(2, "/bin/sh: ", 9) = 9 [pid 8172] write(2, "Syntax error: Unterminated quote"..., 40) = 40 Further investigation with strace revealed that digikam uses dcop to set the background image - something to the effect of: dcop kdesktop KBackgroundIface setWallpaper ... I'm not sure if this is the correct place for the fix, but I found the following code in imageviewer.cpp: QString com=QString("dcop kdesktop KBackgroundIface setWallpaper '%1' %2 >/dev/null 2>/dev/null") .arg(m_loaded_image_url.url()) The problem seems pretty obvious from that snippet. I'll try and submit a patch later. John, The code to set an image as desktop wallpaper is embeded in a kipi-plugin, not in digiKam core. Code is here: http://websvn.kde.org/branches/extragear/kde3/libs/kipi-plugins/wallpaper/plugin_wallpaper.cpp?revision=670712&view=markup Look the method Plugin_WallPaper::setWallpaper() for details Gilles Caulier John, What's news about this file ? Gilles Caulier this plugin is currently disabled yes, because a Plasma do not have yet an interface to push image over background desktop. I don't know yet about KDE 4.3 (can be fixed). DBUS Interface must be called "org.kde.desktop.Background.xml"... Gilles Caulier *** This bug has been marked as a duplicate of bug 217950 *** Fixed with bug #217950 |