[Sidebar] Avoid reloading already loaded module

BUG: 420196

sidebar/SidebarMode.cpp

@@ -436,18 +436,23 @@ void SidebarMode::loadModule( const QModelIndex& activeModule, const QStringList
         return;
     }
 
-    if( !d->moduleView->resolveChanges() ) {
+    MenuItem *mi = activeModule.data(MenuModel::MenuItemRole).value<MenuItem *>();
+
+    if (!mi) {
         return;
     }
 
-    d->moduleView->closeModules();
-
-    MenuItem *mi = activeModule.data(MenuModel::MenuItemRole).value<MenuItem *>();
+    // If we are trying to load a module already open
+    if (d->moduleView->activeModule() && mi->item() == *d->moduleView->activeModule()) {
+        return;
+    }
 
-    if (!mi) {
+    if( !d->moduleView->resolveChanges() ) {
         return;
     }
 
+    d->moduleView->closeModules();
+
     if (homeItem()) {
         d->m_introPageVisible = activeModule == d->categorizedModel->mapFromSource(d->model->indexForItem(homeItem()));
         emit introPageVisibleChanged();