Don't call twice same method

CMakeLists.txt

@@ -69,7 +69,7 @@ set(MAILCOMMON_LIB_VERSION_LIB "5.12.50")
 set(KDEPIM_APPS_LIB_VERSION_LIB "5.12.43")
 set(MESSAGELIB_LIB_VERSION_LIB "5.12.72")
 set(LIBKLEO_LIB_VERSION_LIB "5.12.40")
-set(PIMCOMMON_LIB_VERSION_LIB "5.12.41")
+set(PIMCOMMON_LIB_VERSION_LIB "5.12.42")
 set(LIBKDEPIM_LIB_VERSION_LIB "5.12.40")
 set(LIBKSIEVE_LIB_VERSION_LIB "5.12.40")
 

src/kmail_part.cpp

@@ -130,7 +130,7 @@ void KMailPart::guiActivateEvent(KParts::GUIActivateEvent *e)
 {
     KParts::ReadOnlyPart::guiActivateEvent(e);
     if (e->activated()) {
-        mainWidget->initializeFilterActions();
+        mainWidget->initializeFilterActions(true);
         mainWidget->tagActionManager()->createActions();
         mainWidget->folderShortcutActionManager()->createActions();
         mainWidget->populateMessageListStatusFilterCombo();

src/kmmainwidget.cpp

@@ -4112,7 +4112,7 @@ void KMMainWidget::slotIntro()
 void KMMainWidget::slotShowStartupFolder()
 {
     connect(MailCommon::FilterManager::instance(), &FilterManager::filtersChanged,
-            this, &KMMainWidget::initializeFilterActions);
+            this, [this]() { initializeFilterActions(true); });
     // Plug various action lists. This can't be done in the constructor, as that is called before
     // the main window or Kontact calls createGUI().
     // This function however is called with a single shot timer.
@@ -4134,7 +4134,7 @@ void KMMainWidget::checkAkonadiServerManagerState()
 {
     Akonadi::ServerManager::State state = Akonadi::ServerManager::self()->state();
     if (state == Akonadi::ServerManager::Running) {
-        initializeFilterActions();
+        initializeFilterActions(true);
     } else {
         connect(Akonadi::ServerManager::self(), &ServerManager::stateChanged,
                 this, &KMMainWidget::slotServerStateChanged);
@@ -4144,7 +4144,7 @@ void KMMainWidget::checkAkonadiServerManagerState()
 void KMMainWidget::slotServerStateChanged(Akonadi::ServerManager::State state)
 {
     if (state == Akonadi::ServerManager::Running) {
-        initializeFilterActions();
+        initializeFilterActions(true);
         disconnect(Akonadi::ServerManager::self(), SIGNAL(stateChanged(Akonadi::ServerManager::State)));
     }
 }
@@ -4211,7 +4211,7 @@ void KMMainWidget::clearFilterActions()
 
 void KMMainWidget::clearPluginActions()
 {
-    //KMailPluginInterface::self()->clearPluginActions(QStringLiteral("kmail"), mGUIClient);
+    KMailPluginInterface::self()->clearPluginActions(QStringLiteral("kmail"), mGUIClient);
 }
 
 void KMMainWidget::initializePluginActions()
@@ -4239,9 +4239,11 @@ QAction *KMMainWidget::filterToAction(MailCommon::MailFilter *filter)
 }
 
 //-----------------------------------------------------------------------------
-void KMMainWidget::initializeFilterActions()
+void KMMainWidget::initializeFilterActions(bool clearFilter)
 {
-    clearFilterActions();
+    if (clearFilter) {
+        clearFilterActions();
+    }
     mApplyFilterActionsMenu->menu()->addAction(mApplyAllFiltersAction);
     mApplyFilterFolderActionsMenu->menu()->addAction(mApplyAllFiltersFolderAction);
     mApplyFilterFolderRecursiveActionsMenu->menu()->addAction(mApplyAllFiltersFolderRecursiveAction);

src/kmmainwidget.h

@@ -169,7 +169,7 @@ public:
 
     QString fullCollectionPath() const;
 
-    void initializeFilterActions();
+    void initializeFilterActions(bool clearFilter);
     /** Clear and create actions for marked filters */
     void clearFilterActions();
     /**

src/kmmainwin.cpp

@@ -163,9 +163,9 @@ void KMMainWin::slotUpdateGui()
     applyMainWindowSettings(KMKernel::self()->config()->group("Main Window"));
 
     // plug dynamically created actions again
-    mKMMainWidget->initializeFilterActions();
+    mKMMainWidget->initializeFilterActions(false);
     mKMMainWidget->tagActionManager()->createActions();
-    mKMMainWidget->initializePluginActions();
+    //FIXME mKMMainWidget->initializePluginActions();
 }
 
 void KMMainWin::setupStatusBar()