Version: (using KDE KDE 3.2.2) Installed from: Debian testing/unstable Packages 09:28:27 <[MadCoder]> if you right click in konqui on a rss feed 09:28:37 <[MadCoder]> the 'add to akregator' don't work 09:33:52 <lippel> [MadCoder]: right, it doesnt add anything... was mail-reported to me by John Belmonte : Hello, When I use "add feed to aKregator" from Konqueror, the feed does not appear in akregator. Are you aware of any such problem? (bugs.kde.org is down so I can't check upstream.) -John
CVS commit by osterfeld: fixes bug #84295. Search in DCOP interface for "akregatorIface" instead of "akregatorpart". CCMAIL: 84295-done@bugs.kde.org M +4 -3 akregatorplugin.cpp 1.6 --- kdenonbeta/akregator/src/konq_plugin/akregatorplugin.cpp #1.5:1.6 @@ -155,5 +155,5 @@ void AkregatorMenu::slotAddFeed() for (clientIt = akregatorObjs.begin(); (clientIt != akregatorObjs.end()) ; ++clientIt ) { - if ((*clientIt).contains("akregatorpart", false) != 0) + if ((*clientIt).contains("akregatorIface", false) != 0) { akregatorObj = *clientIt; @@ -162,5 +162,6 @@ void AkregatorMenu::slotAddFeed() } } - if (akregatorObj.isNull()) return; + if (akregatorObj.isNull()) + return; KFileItemListIterator it( m_list );
I'm on KDE 3.2.1, running CVS of Akregator but this is not entirely working for me. Add Feed To Akregator ONLY works when Akregator is not running. A new instance gets started and the feed gets added. If Akregator is already running. Nothing happens. I've investigated it and there's a silly little error in the DCOP code you use. In src/konq_plugin/akregatorplugin.cpp on line 157 you have this: if ((*clientIt).contains("akregatorIface", false) != 0) According to kdcop, akregator only exports aKregatorIface, with a capitalized K. I suppose this makes the contains() method case sensitive. Turning it into: if ((*clientIt).contains("aKregatorIface", false) != 0) Fixes it. Hope this helps.
CVS commit by sashmit: add feed if akregator is running thanks mathieu kooiman for the fix CCMAIL: 84295@bugs.kde.org M +1 -1 akregatorplugin.cpp 1.7 --- kdenonbeta/akregator/src/konq_plugin/akregatorplugin.cpp #1.6:1.7 @@ -155,5 +155,5 @@ void AkregatorMenu::slotAddFeed() for (clientIt = akregatorObjs.begin(); (clientIt != akregatorObjs.end()) ; ++clientIt ) { - if ((*clientIt).contains("akregatorIface", false) != 0) + if ((*clientIt).contains("aKregatorIface", false) != 0) { akregatorObj = *clientIt;