From de3ae0ddca41594f4e85214ce637c99507ee2c96 Mon Sep 17 00:00:00 2001
From: mzn <mzn@opencascade.com>
Date: Wed, 15 Sep 2010 11:36:06 +0000
Subject: [PATCH] Correction: remove destroyed actions from the actions list.

---
 src/SUIT/SUIT_ShortcutMgr.cxx | 23 ++++++++++++++++++++---
 src/SUIT/SUIT_ShortcutMgr.h   |  8 ++++++--
 2 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/src/SUIT/SUIT_ShortcutMgr.cxx b/src/SUIT/SUIT_ShortcutMgr.cxx
index ab5afccbc..44eb8a950 100644
--- a/src/SUIT/SUIT_ShortcutMgr.cxx
+++ b/src/SUIT/SUIT_ShortcutMgr.cxx
@@ -119,9 +119,11 @@ void SUIT_ShortcutMgr::processAction( QtxAction* action )
  
   if ( !shortcutActionName.isEmpty() ) {
     // Add action to the actions map
-    QList<QtxAction*> actions = myShortcutActions.values( shortcutActionName );
-    if ( !actions.contains(action) )
-      myShortcutActions.insertMulti( shortcutActionName, action );
+    if ( !myShortcutActions.contains( shortcutActionName, action ) ) {
+      myShortcutActions.insert( shortcutActionName, action );
+      connect( action, SIGNAL( destroyed( QObject* ) ), 
+	       this, SLOT ( onActionDestroyed( QObject* ) ) );
+    }
 
     QKeySequence keySeq = getShortcutByActionName( shortcutActionName );
     action->setShortcut( keySeq );
@@ -168,3 +170,18 @@ void SUIT_ShortcutMgr::updateShortcuts()
     action->setShortcut( keySeq );
   }
 }
+
+/*!
+  \brief Called when the corresponding action is destroyed.
+  
+  Removes destroyed action from the actions list.
+
+  \param obj action being destroyed
+*/
+void SUIT_ShortcutMgr::onActionDestroyed( QObject* obj )
+{
+  QtxAction* anAction = (QtxAction*)obj;
+  
+  if ( anAction )
+    myShortcutActions.remove( anAction->shortcutActionName(), anAction );
+}
diff --git a/src/SUIT/SUIT_ShortcutMgr.h b/src/SUIT/SUIT_ShortcutMgr.h
index e620db8e6..eafd854e2 100644
--- a/src/SUIT/SUIT_ShortcutMgr.h
+++ b/src/SUIT/SUIT_ShortcutMgr.h
@@ -26,7 +26,7 @@
 #include "SUIT.h"
 
 #include <QObject>
-#include <QMap>
+#include <QMultiMap>
 
 class QtxAction;
 
@@ -42,6 +42,7 @@ class QKeySequence;
 */
 class SUIT_EXPORT SUIT_ShortcutMgr: public QObject 
 {
+  Q_OBJECT
 public:
   static void Init();
   static SUIT_ShortcutMgr* getShortcutMgr();
@@ -53,6 +54,9 @@ protected:
   SUIT_ShortcutMgr();
   virtual ~SUIT_ShortcutMgr();
 
+private slots:
+  void onActionDestroyed( QObject* );
+
 private:
   virtual bool eventFilter( QObject* o, QEvent* e );
 
@@ -61,7 +65,7 @@ private:
 
 private:
   static SUIT_ShortcutMgr* myShortcutMgr;
-  QMap<QString, QtxAction*> myShortcutActions;
+  QMultiMap<QString, QtxAction*> myShortcutActions;
 };
 
 #if defined WIN32
-- 
2.39.2