From c7f1f65ed85059462c2de8bd35abf5a2005d4aa9 Mon Sep 17 00:00:00 2001 From: Thomas McGuire Date: Mon, 20 Aug 2007 19:16:22 +0000 Subject: [PATCH] Fix double connect triggered by sharing the actions in two menus. Better use two actions instead. I caused this in the transport changes some minutes ago. svn path=/trunk/KDE/kdepim/; revision=702530 --- kmcomposewin.cpp | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/kmcomposewin.cpp b/kmcomposewin.cpp index 1c4176275..3b92886bb 100644 --- a/kmcomposewin.cpp +++ b/kmcomposewin.cpp @@ -1154,10 +1154,12 @@ void KMComposeWin::getTransportMenu() availTransports = TransportManager::self()->transportNames(); QStringList::Iterator it; for ( it = availTransports.begin(); it != availTransports.end() ; ++it ) { - QAction *action = new QAction( (*it).replace( "&", "&&" ), mActNowMenu ); - action->setData( TransportManager::self()->transportByName( *it )->id() ); - mActNowMenu->addAction( action ); - mActLaterMenu->addAction( action ); + QAction *action1 = new QAction( (*it).replace( "&", "&&" ), mActNowMenu ); + QAction *action2 = new QAction( (*it).replace( "&", "&&" ), mActLaterMenu ); + action1->setData( TransportManager::self()->transportByName( *it )->id() ); + action2->setData( TransportManager::self()->transportByName( *it )->id() ); + mActNowMenu->addAction( action1 ); + mActLaterMenu->addAction( action2 ); } }