[Bug 154970] Launcher Menu starts application twice

[Sebastian Sauer]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=154970         




------- Additional Comments From mail dipe org  2008-01-03 05:03 -------
Thanks Gabriel :)

Seems something on my setup doesn't like that at all. Every desktop-file which has the core-category isn't displayed within any of the launchers. Even hard-coding an item doesn't seem to work for me :-/

Anyway, to provide some more input in the hope somebody got it fixed before I was able to find a way to reproduce it;

Within the file http://websvn.kde.org/trunk/KDE/kdebase/workspace/plasma/applets/kickoff/simpleapplet/simpleapplet.cpp?view=markup the method
MenuLauncherApplet::actionTriggered(QAction *action)
does handle the "menuitem got executed" event. For that it just tries to redirect the call to the proper view (the kmenu may display multiple MenuView instances where we use e.g. one view for the ApplicationModel and another view for the FavoritesModel - all of them got them merged together into the single KMenu Launcher). The MenuView itself is located at http://websvn.kde.org/trunk/KDE/kdebase/workspace/plasma/applets/kickoff/simpleapplet/menuview.cpp?view=markup and there we call the MenuView::actionTriggered method.
Now I can imagine that the problem is, that within the constructor of that class we do connect(this,SIGNAL(triggered(QAction*)),this,SLOT(actionTriggered(QAction*)));

Without testing all that, I could imagine that following patch fixes the problem. But I don't know if it really does nor am I able to check if it introduces any sideeffects :-/

Index: simpleapplet/menuview.cpp
===================================================================
--- simpleapplet/menuview.cpp   (revision 755671)
+++ simpleapplet/menuview.cpp   (working copy)
 @ -75,7 +75,7  @
     : KMenu(parent)
     , d(new Private(this))
 {
-    connect(this,SIGNAL(triggered(QAction*)),this,SLOT(actionTriggered(QAction*)));
+    //connect(this,SIGNAL(triggered(QAction*)),this,SLOT(actionTriggered(QAction*)));
 }
 MenuView::~MenuView()
 {

More next days once I got my local prob sorted out.